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.

Sign up for GitHub

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

Jump to bottom

Fix support for STAC and replace bands_requested and bbox type for list instead of string. #32

Merged

mpelchat04 merged 5 commits into NRCan:main from mpelchat04:bbox

Nov 13, 2024

Collaborator

mpelchat04 commented Oct 31, 2024

Fix support for STAC item as input.
Add tests for STAC support.
Add tests for bbox. [BUG] BBOX parameter not working properly. #25
Replace comma separated strings by list for parameters bands_requested and bbox. Closes [Bug] - Bands_requested does not accept list. #31
Fix import issue for rioxarray.
Fix import issue for rasterio.window from_origin method.

Turgeon-Pelchat added 4 commits

October 31, 2024 13:19


          fix import + fix STAC usage + STAC test

0b991a9


          bands_requested and bbox as list instead of comma separated string

852825f


          yaml diff fix.

468ebb5


          doc typo

bcf40bd

mpelchat04 requested review from valhassan and jfbourgon

October 31, 2024 19:17

valhassan approved these changes

View reviewed changes

Collaborator

valhassan left a comment

easy review!


          bbox param fix

3d362b8

mpelchat04 merged commit 9e02167 into NRCan:main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet