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

Gallery examle no10 downloads large amount of data #2222

Closed
bsipocz opened this issue Nov 19, 2021 · 2 comments · Fixed by #2562
Closed

Gallery examle no10 downloads large amount of data #2222

bsipocz opened this issue Nov 19, 2021 · 2 comments · Fixed by #2562

Comments

@bsipocz
Copy link
Member

bsipocz commented Nov 19, 2021

The "Retrieve Hubble archival data of M83 and make a figure" example (https://astroquery.readthedocs.io/en/latest/gallery.html#example-10) downloads an awful lot of data (I've stopped the run at 6+GB, it wasn't clear how much longer would it go), which is totally unnecessary for showcasing an astroquery functionality. It would be great to rephrase or even totally exchange that example to involve only a small amount of data.

This is related to #863, #1905 and even may be even #307 (the latter only if we decide to go with a sphinx gallery approach for the examples)

cc @jaymedina @tomdonaldson

@keflavich
Copy link
Contributor

We could remove & deprecate the astroquery gallery in favor of examples that use astroquery on learn. All of these use it:
image

I can't find a way to link to learn.astropy.org with search filtered by astroquery, but that seems more sustainable than the gallery.

So I suggest we make this Issue a checklist for transitioning examples to learn?

@bsipocz
Copy link
Member Author

bsipocz commented Nov 19, 2021

I'm OK with the idea, though it's not clear how stable and reliable that page is in practice.
But basically, my goal is to remove all broken code from this repo. And if examples keep failing somewhere downstream then it's not my problem any more :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants