-
Notifications
You must be signed in to change notification settings - Fork 25
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
Maybe add more features to the condition parameter in QueryATNF? #68
Comments
Thanks for the suggestion. It should be pretty easy to implement the SURVEY and DISCOVERY functions, so I will try and add them in soon. It will take a bit more effort to add range properly and I don't really have the time at the moment. I am more than happy to take pull requests for any code changes though! |
I've added the survey and discovery functions. I'll make a new release that's available via pip and conda when I've merged another issue about the references. |
I would like to learn how to use the astroquery vizier to download two catalogs, a pulsar catalog and a Fast Radio Bursts catalog. Would you help me? |
Hi @renato10nr, I'm afraid I have no experience downloading VizieR catalogs via astroquery, so can't be of much help. From the looks of things VizieR's pulsar catalogs are those extracted from individual papers, whereas this software (psrqpy) gets the catalog of all pulsars as held in the ATNF Pulsar Catalog. |
You have already added support for querying the catalogue using logical conditions, but some things are still not supported, such as the RANGE function, or specifying a range of RA/DEC using the HMS/DMS functions or h/d keywords. It would be nice if this feature was expanded to include these functions, along with the SURVEY, DISCOVERY functions. All the functions I just mentioned are (as you probably know) described here, in section 2.3.
The text was updated successfully, but these errors were encountered: