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

SDSS DR17 radial search API #2450

Closed
pllim opened this issue Jun 22, 2022 · 6 comments
Closed

SDSS DR17 radial search API #2450

pllim opened this issue Jun 22, 2022 · 6 comments

Comments

@pllim
Copy link
Member

pllim commented Jun 22, 2022

How do I use astroquery.sdss to access http://skyserver.sdss.org/dr17/SearchTools/radial programmatically? Thanks!

cc @camipacifici @javerbukh

@bsipocz
Copy link
Member

bsipocz commented Jun 22, 2022

The usual astroquery API for query_region() does that.

It's not the exact same SQL but close enough (and if anyone wants to fix to send the radius up to the SQL server rather than hand calculate a box, PRs are welcome, see #1088 with suggestion for the fix, and discussion in #585 and possibly some other issues, too).

If the exact match is important, then I would suggest directly using query_sql.

@bsipocz bsipocz closed this as completed Jun 22, 2022
@bsipocz
Copy link
Member

bsipocz commented Jun 22, 2022

Oh, and we know that the documentation for the sdss module is somewhat lacking, PRs to extend it are more than welcome (and there are gazillion examples and coverage for different queries in the tests, those can be used as templates).

@pllim
Copy link
Member Author

pllim commented Jun 22, 2022

Belated cc to @svolfman

@pllim
Copy link
Member Author

pllim commented Jun 22, 2022

Ah, so one needs to set this first before using DR17?

default_release = _config.ConfigItem(14, 'Default SDSS data release.')

@bsipocz
Copy link
Member

bsipocz commented Jun 22, 2022

They can, but there is a data_release kwarg, too that should do the deed.

@pllim
Copy link
Member Author

pllim commented Jun 22, 2022

Thanks for the quick response!

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

No branches or pull requests

2 participants