Skip to content

Comments

Replace sentinelsat with eodag#78

Merged
rustamdantia merged 3 commits intomainfrom
replace-sentinelsat-with-eodag
Feb 17, 2025
Merged

Replace sentinelsat with eodag#78
rustamdantia merged 3 commits intomainfrom
replace-sentinelsat-with-eodag

Conversation

@rustamdantia
Copy link
Collaborator

@rustamdantia rustamdantia commented Nov 20, 2024

  • Replace the Sentinel-2 downloader based on the deprecated sentinelsat library with a downloader based on the much powerful eodag library. This allows for downloading various product types from several providers/rs data sources with a single unified API.
  • I couldn't get the pipeline for python 3.8 to pass with the new eodag dependency, so geographer will now only support python >= 3.9.
  • Replace custom __init__ logic in DSCreatorFromSource with pydantic validators.

@rustamdantia rustamdantia force-pushed the replace-sentinelsat-with-eodag branch 2 times, most recently from 0aaa468 to d07d874 Compare November 20, 2024 23:32
@rustamdantia rustamdantia marked this pull request as draft November 20, 2024 23:33
@rustamdantia rustamdantia force-pushed the replace-sentinelsat-with-eodag branch 9 times, most recently from 0755b19 to 856c990 Compare November 21, 2024 11:08
@rustamdantia rustamdantia force-pushed the replace-sentinelsat-with-eodag branch 2 times, most recently from 8ac5858 to cc38c89 Compare November 21, 2024 12:08
@rustamdantia rustamdantia changed the title WIP Replace sentinelsat with eodag Replace sentinelsat with eodag Nov 21, 2024
@rustamdantia rustamdantia force-pushed the replace-sentinelsat-with-eodag branch 5 times, most recently from 0ece6dd to 00a6ecc Compare November 21, 2024 22:42
@rustamdantia rustamdantia force-pushed the replace-sentinelsat-with-eodag branch from 3896f74 to 93aae32 Compare December 9, 2024 11:28
@rustamdantia rustamdantia marked this pull request as ready for review December 9, 2024 11:48
@rustamdantia rustamdantia force-pushed the replace-sentinelsat-with-eodag branch from 6ded720 to 292457a Compare January 2, 2025 13:24
Comment on lines +322 to +335
log.error(msg, location_name, extracted_product_file_name)
raise RuntimeError(
msg % (msg, location_name, extracted_product_file_name)
)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the main idea is that you would have one single base class for Geographer errors so that your users can have a simple "fallback" way to handle all errors coming from the library. And then you can progressively define more specific error types (as can your users in their error handling code).

@rustamdantia rustamdantia force-pushed the replace-sentinelsat-with-eodag branch 4 times, most recently from 018ec13 to 4e90c16 Compare February 17, 2025 22:30
@rustamdantia rustamdantia force-pushed the replace-sentinelsat-with-eodag branch from 4e90c16 to b6dc5ca Compare February 17, 2025 22:34
@rustamdantia rustamdantia merged commit f3d1c81 into main Feb 17, 2025
5 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.

2 participants