-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fix the Links of Spectra for Suarez22 #347
Comments
Please be more specific about what "doesn't work" mean and give a couple examples. |
The issue lies in that the sources which have a plus in the name like 2MASS J21481633+4003594 have the url name of https://bdnyc.s3.amazonaws.com/IRS/2148%252B4003_IRS_spectrum.fits which does not work because the actual working link is https://bdnyc.s3.amazonaws.com/IRS/2MASS+J21481633%2B4003594.fits. This is the google doc link to the correct spectra urls : https://docs.google.com/spreadsheets/d/1ojsNF59GvdFhf-lgwJaCwiHPfaRx4fmCpyzKEonrVbw/edit?usp=sharing |
We should write a test which finds these problems before we fix them. Issue opened #369 |
another example: https://simple-bd-archive.org/load_solo/2MASS%20J12130336-0432437 |
There are 149 URLs giving 404s: |
Progress. After #404 , here are the ones left: assert len(broken_urls) == 149, f'found {len(broken_urls)} broken spectra urls: {broken_urls}, {codes}' |
The links for the spectra for the Suarez22 ingest don't work and need to be fixed to have the correct ones.
The text was updated successfully, but these errors were encountered: