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

Feature request: download tif's if needed in a composite #1362

Closed
raybellwaves opened this issue Sep 14, 2020 · 3 comments · Fixed by #1513
Closed

Feature request: download tif's if needed in a composite #1362

raybellwaves opened this issue Sep 14, 2020 · 3 comments · Fixed by #1513

Comments

@raybellwaves
Copy link
Contributor

Apologies this is not a MCVE but I think it's ok.

When I do scn.load(['night_ir_with_background_hires']). I get RasterioIOError: BlackMarble_2016_3km_geo.tif: No such file or directory. I see here

filename: BlackMarble_2016_3km_geo.tif

and I see it's looking for a local file BlackMarble_2016_01deg_geo.tif.

After some google I see the file can be downloaded from https://neo.sci.gsfc.nasa.gov/archive/blackmarble/2016/global/ (I believe).

It would be good if a user who does scn.load(['night_ir_with_background_hires']) need not go and download the file but have the API do it.

An example, this is implemented in cartopy when a user requests coastline of different resolution e.g.
ax.coastlines(resolution='10m'). I don't know exactly how they implement but could take inspiration from there.
e.g. https://github.com/SciTools/cartopy/blob/8d11866c06a465764063d9f7fcfa70e8476089e0/tools/feature_download.py

@djhoese
Copy link
Member

djhoese commented Sep 14, 2020

Agreed. This is a known issue that kind of got away from us. This was one of those "oh it would be cool if we could use a static image as a background" and then never actually did it "the right way". Thanks for filing the issue.

@raybellwaves
Copy link
Contributor Author

However, it's a big(ish) file (64.4 Mb) so couldn't prompt for a Y/N for the download

@djhoese
Copy link
Member

djhoese commented Sep 14, 2020

I'd say either we raise an exception saying you need to run a utility function to download the file or automatically download it or both and have a global config option to control what happens.

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 a pull request may close this issue.

2 participants