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

enhance ee.Image.fullLike #266

Merged
merged 1 commit into from
May 17, 2024
Merged

enhance ee.Image.fullLike #266

merged 1 commit into from
May 17, 2024

Conversation

fitoprincipe
Copy link
Member

ee.Projection not only stores the CRS but also the nominal scale, and although the CRS is usually constant across all bands, is not the same for the nominal scale (example here). So, one of the modification of this PR is to copy each band's projection to the image generated by ee.Image.fullLike.

In the previous version of fullLike the footprint of the original images was passed to the resulting image. Now the user can choose whether to copy it or not, giving more flexibility to the user.

@fitoprincipe
Copy link
Member Author

test failure is weird since the failing test is not using the modified method at all 🤔

@12rambau
Copy link
Member

12rambau commented May 16, 2024

The test failure is related to GEE slightly changing it's rounding method all the time, you must nuke the nox folder from your side and start the complete test suit again to regenerate the file. I would like to have a better way of testing this with pytest-gee but never found the time to work on it yet.

@12rambau 12rambau merged commit 081864c into main May 17, 2024
8 of 10 checks passed
@12rambau 12rambau deleted the fix-full-like branch May 17, 2024 06:40
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.

None yet

2 participants