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

Add ephemerides benchmarks #14

Open
cdeil opened this issue Dec 3, 2012 · 6 comments
Open

Add ephemerides benchmarks #14

cdeil opened this issue Dec 3, 2012 · 6 comments

Comments

@cdeil
Copy link
Member

cdeil commented Dec 3, 2012

Some day we want to have astropy support coordinate computations for solar system bodies like the sun, moon and planets (called ephemerides):
https://groups.google.com/d/topic/astropy-dev/jjH0dtdByxk/discussion

When this effort starts we should add checks against other tools:
http://rhodesmill.org/pyephem/quick.html#bodies
http://ssd.jpl.nasa.gov/?ephemerides

@eteq
Copy link
Member

eteq commented Dec 3, 2012

+1 to this!

@astrojuanlu
Copy link
Member

Prior to the benchmarks, is there an issue or PR in the core astropy repo to track progress in adding ephemerides computations?

@cdeil
Copy link
Member Author

cdeil commented Jul 5, 2013

I'm don't know exactly what is going on with coordinates and ephemerides in astropy.
I see you have started something here.

@eteq will know what the status is and if you are interested in contributing to astropy.
Possibly a discussion on the astropy-dev mailing list followed by an astropy-api proposal.

@astrojuanlu
Copy link
Member

Well, actually in that project you pointed out there's already some bare bones ephemerides calculations based on some numerical formulas which compute the osculating orbital elements. It worked for me because I had to perform some medium-precision computations, but I haven't compared them with, say, the JPL ephemerides. Other than that, it might be preferable to have RA/dec ephemerides for example? I have followed only one course on Orbital Mechanics so I might probably miss some important details.

FWIW, I collected some of the relevant projects and ideas in this issue.

@eteq
Copy link
Member

eteq commented Jul 8, 2013

@Juanlu001 - there have not been any explicit plans to include ephemerides in astropy yet, mainly because we're working on re-vamping coordinates for 0.3, and of course anything with ephemerides will need to make extensive use of coordinates. But it's definitely something we will want in astropy at some point hopefully in the not-too-distant future. Feel free to make an issue to do this in the main astropy repo, if you like.

(Also, if you don't know about this, @Juanlu001, note that you don't need to give full links to issues on github - if you just type Pybonacci/poliastro#1, github is clever enough to recognize the link automatically: poliastro/poliastro#1 and it also creates indications in the issue that it has been commented on here)

There has been some discussion of this on astropy-dev in rather vague terms - the main points to consider are that the JPL ephemerides should be downloaded using the astropy download/caching tools, the result definitely should be astropy.coordinates.<something> objects, and where relevant, the SOFA routines (soon to be ERFA) for earth/sun/moon location and the major planets should be used instead of "home-grown" approximations.

@astrojuanlu
Copy link
Member

Thanks for the information and the clarification. I have not been following dev-related discussions but I see in the repo issues that you are working hard on coordinates, and on the other hand right now I am basically gathering fundamental knowledge about the matter, so as of today I think I will wait a while to open that issue.

Anyway, I will go on learning and keep an eye on this just in case I can be of any help when the time comes.

Kudos for the great work, astropy is my favorite Python project in terms of organization and code quality by far :)

(And thanks for the tip about issues links!)

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

4 participants