MAINT: switch pyvo obscore query with astroquery #125
Conversation
tutorials/spherex/spherex_intro.md
Outdated
| * `'spherex_qr'` -- the Quick Release Observations, for more information see [SPHEREx archive documentation at IRSA](). | ||
| * `'spherex_qr_cal'` -- the Quick Release Calibration products. | ||
| * `'spherex_qr_deep'` -- |
There was a problem hiding this comment.
I'm guessworking here, @vandesai1 please provide guidence here.
There was a problem hiding this comment.
@troyraen - I'm still missing on this definitions/description
tutorials/spherex/spherex_intro.md
Outdated
| The `'access_url'` column is particularly important because it tells you how to access the data. Let's look at the `'access_url'` value for the first row: | ||
|
|
||
| ```{code-cell} ipython3 | ||
| results['access_url'][0] |
There was a problem hiding this comment.
The PR is draft as this line currently returns an incomplete HTTPS url rather than a datalink.
There was a problem hiding this comment.
This appears to have been fixed, at least in the sense that it is now returning a full url instead of a relative path.
>>> results = Irsa.query_sia(pos=(coord, search_radius), collection='spherex_qr')
>>> results['access_url'][0]
'https://irsa.ipac.caltech.edu/ibe/data/spherex/qr/level2/2025W20_2D/l2b-v12-2025-176/6/level2_2025W20_2D_0444_1D6_spx_l2b-v12-2025-176.fits'There was a problem hiding this comment.
Thanks, that's great news!
bsipocz
force-pushed
the
MAINT_spherex_switch_astroquery
branch
from
88bd7da to
58143c5
Compare
|
@troyraen - this should be good to go now, I removed the DL explanations for now as we don't ship SIA with DL atm. (I expect some possibly unrelated failures, as something popped up in cron yesterday and I haven't yet looked into them closely) |
tutorials/spherex/spherex_intro.md
Outdated
| import pyvo | ||
| from astroquery.ipac.irsa import Irsa | ||
| from pyvo.dal.adhoc import DatalinkResults |
There was a problem hiding this comment.
Remove this and also remove pyvo from the install cell.
| from astropy.coordinates import SkyCoord | ||
| import pyvo | ||
| from astroquery.ipac.irsa import Irsa |
There was a problem hiding this comment.
Add astroquery to the install cell.
Co-authored-by: Troy Raen <[email protected]>
|
As I mentioned above, the rendering issues are unrelated, so I go ahead and look into them separately. |
2 participants
This is TBD as IRSA SIA is not just yet returning datalinks