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

python API based Google Earth Engine download #88

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

Aryal007
Copy link
Collaborator

@Aryal007 Aryal007 commented Feb 3, 2021

removed legacy code to download images by country/year borders and added python based google earth engine api to download images given

  1. Image Ids and, to download image for training
  2. Wrs path/row for a region, between start date and end date, to download new images for a different time for inference

@krisrs1128
Copy link
Owner

Hi Bibek, I made a couple small changes, but this looks great to me. Thank you for preparing the script, I like it much more than using the GEE editor.

Two questions before merging, though,

  1. Is there are reason we're using EPSG:32644? I suspect we'll end up warping it to 4326 anyways, but maybe in this region there is a reason to prefer the other CRS.
  2. Are you sure it's okay to convert to .float() at this stage? I noticed that in our current downloads, the LE7 bands are capped at 255, when really they should reach a bit higher.

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