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
Use jyear format instead of decimalyear in apply_space_motion example #11023
Conversation
Hmm @pllim the GH actions labeler got a little presumptuous here in removing the docs label! |
Wow, I didn't know it can remove a label! I'll have to investigate. Sorry for the trouble and thanks for letting me know. |
I wonder if it is a race condition that is not handled properly by that action. Does not seem like something I can fix on our side. I opened actions/labeler#112 . In the meantime, try apply your own labels after the action is done. Sorry for the inconvenience. 😬 |
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.
Thanks!
@adrn - looks all OK, but shouldn't this be backported to LTS? Or was the example not there? |
Oh yes, likely so - I believe this was there and I just mixed up versions! |
Use jyear format instead of decimalyear in apply_space_motion example
Use jyear format instead of decimalyear in apply_space_motion example
Use jyear format instead of decimalyear in apply_space_motion example
Use jyear format instead of decimalyear in apply_space_motion example
This fixes #11009
As @tboch uncovered and @eerovaher figured out, the example fixed in this PR incorrectly used the decimalyear time format for parsing the Gaia epoch '2015.5', which lead to some confusion when comparing
apply_space_motion()
to other epoch propagation algorithms.