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

[Depreciation Warning] : landsat_c1_to_c2 #517

Closed
thekester opened this issue Jul 12, 2024 · 4 comments
Closed

[Depreciation Warning] : landsat_c1_to_c2 #517

thekester opened this issue Jul 12, 2024 · 4 comments

Comments

@thekester
Copy link
Contributor

thekester commented Jul 12, 2024

Hello, i got that DeprecationWarning when i try to execute the code

Attention required for LANDSAT/LT05/C01/T1_TOA! You are using a deprecated asset.
To ensure continued functionality, please update it by July 1, 2024.
Learn more: https://developers.google.com/earth-engine/landsat_c1_to_c2

warnings.warn(warning, category=DeprecationWarning)

landsatdepreciationwarning

in SDS_download.py eight occurrences of T1_TOA in check_images_available function and get_image_info

@kvos
Copy link
Owner

kvos commented Jul 25, 2024

did you specify inputs['landsat_collection'] = 'C02'? if you specify collection 1 you should not get that warning, if you still do let me know. I want to keep collection 1 in there as some people may have extracted shorelines a while back and the only way they can reproduce their results is using collection 1.

@thekester
Copy link
Contributor Author

thekester commented Jul 25, 2024

When I use collection = 'C01', I get a DeprecationWarning. However, if I use collection = 'C02', I do not get this DeprecationWarning. I understand you want to keep collection 1, but unfortunately after checking this guide, it indicates that the Landsat1 collection will be removed in the second half of 2024. You can check the Landsat collection status here
image

@kvos
Copy link
Owner

kvos commented Jul 25, 2024

yes I think they removed Collection 1 this week. Will adjust the code soon.

@thekester
Copy link
Contributor Author

Closed by #526

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

No branches or pull requests

2 participants