carpentries-incubator / geospatial-python

Introduction to Geospatial Raster and Vector Data with Python
https://carpentries-incubator.github.io/geospatial-python/
Other
159 stars 57 forks source link

Update 05-access-data.md #139

Closed abarciauskas-bgse closed 1 year ago

abarciauskas-bgse commented 1 year ago

Suggest using earthaccess library

rbavery commented 1 year ago

@abarciauskas-bgse thanks for this suggestion! It's been on my mind to update the lesson to add this package. We'll also need to add it to the setup instructions, yaml, and would be good to just use this rather than suggesting it. I'm going to be making updates to ep 5 as a part of #138 so I can also make the above changes so that accessing CMR is simpler. First wanted to check though, did you have any uncommitted changes on this PR?

I think the use of this would look similar to this tutorial, where we persist the login so folks don't need to make the .netrc: https://nasa-impact.github.io/ml-pipeline/docs/01_datapipelines_with_torchdata.html

abarciauskas-bgse commented 1 year ago

I didn't have any uncommitted changes, but your suggestions to improve upon this suggestion make sense to me. Is it ok if I try to add them and you can help me ensure it's compatible + conformant with the rest of the repository?

abarciauskas-bgse commented 1 year ago

Closing in favor of https://github.com/carpentries-incubator/geospatial-python/pull/140