astroquery icon indicating copy to clipboard operation
astroquery copied to clipboard

contribution idea: Planetary Ring Node

Open emolter opened this issue 3 years ago • 3 comments

Hi there,

I'd like to add the functionality to query the ephemeris tools in SETI's Planetary Ring Node. This would be useful because the Horizons query tool does not provide ephemeris information on rings like the opening angle, periapsis, or ascending node, and it's helpful in my research to be able to quickly find this information. I read the contributing guidelines and it looks like everything I want to do can be implemented straightforwardly with the Astroquery tools.

Would this contribution be of interest?

Thanks very much for your response, Ned

emolter avatar Apr 06 '22 19:04 emolter

@emolter - I'm not familiar with this science field to know how many potential users would be for such a module, but @mkelley, the maintainer of the solarsystem modules may be able to comment on it. I see no reasons why this could not be a submodule under astroquery.solarsystem, maybe an astroquery.solarsystem.pds?

bsipocz avatar Apr 06 '22 19:04 bsipocz

Thanks @bsipocz ! I went ahead and started building this. I submitted a draft pull request here. At this stage I have a minimum working example written that can make and parse a basic query and return the relevant data as astropy tables. I tested it for each of the six ring systems in the solar system.

I read the astroquery API specification page and the contributing guidelines, but I still have a few questions about how exactly various things should be implemented to fit best into astroquery style. What is the best avenue to get those questions answered?

emolter avatar Apr 08 '22 21:04 emolter

but I still have a few questions about how exactly various things should be implemented to fit best into astroquery style. What is the best avenue to get those questions answered?

If you would leave comments on the PR code, I think that would be the best way to start those discussions

bsipocz avatar Apr 11 '22 22:04 bsipocz