Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Archive] replace column 'bbox' with 'geometry' #296

Merged

Conversation

MarkusZehner
Copy link
Contributor

@MarkusZehner MarkusZehner commented Mar 28, 2024

replaces the column bbox with geometry containing the footprint geometry instead of the bounding box. Import of old databases can be done with import_outdated.

@johntruckenbrodt
Copy link
Owner

I have added some more commits removing one test database (to reduce data volume) and avoid database modification. @MarkusZehner the newly added databases contained absolute paths to your local Sentinel-1 scenes. the test database now contains a relative path so that the database entry is not deleted when cleanup=True because it could not be found (which caused the database modification) and the scene can be identified relative to the database file during import_outdated. Before, the scene entry was deleted while opening the database and there was nothing left to do in import_outdated causing the tests to succeed on AppVeyor.

@johntruckenbrodt johntruckenbrodt changed the title updated geometry column in archive [Archive] replace column 'bbox' with 'geometry' Apr 11, 2024
@johntruckenbrodt johntruckenbrodt merged commit 6a2968f into johntruckenbrodt:main Apr 16, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Archive] use geometry instead of bounding box
2 participants