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
Fix bug in get_horizons_coord time dictionary arg #7106
Conversation
77d89b2
to
0b2355f
Compare
0b2355f
to
80555bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add a line to convert the returned obstime back to UTC when a dictionary is input? That is, add a line after this one:
sunpy/sunpy/coordinates/ephemeris.py
Line 329 in 64419c2
obstime = parse_time(result['datetime_jd'], format='jd', scale='tdb') |
Co-authored-by: Albert Y. Shih <ayshih@gmail.com>
Co-authored-by: Albert Y. Shih <ayshih@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
PR Description
Fix small bug in get_horizons_coord when time is specified using the dictionary format. Update code to pass full time spec (including milliseconds) to astroquery.
Closes #7105