-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
TST: fix compatibility with numpy 2 (copy semantics) in a coordinates test #16385
TST: fix compatibility with numpy 2 (copy semantics) in a coordinates test #16385
Conversation
9ae4b29
to
2ec124b
Compare
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
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.
Well, if that's all you missed - pretty good! Let's get it in.
@mhvk I don't see a backport PR for this. Can you ping the bot here ? (I assume I don't have rights to do it myself) |
@meeseeksdev backport to 6.0.x |
Something went wrong ... Please have a look at my logs. It seems that the branch you are trying to backport to does not exist. |
@meeseeksdev backport to 6.1.x |
Something went wrong ... Please have a look at my logs. It seems that the branch you are trying to backport to does not exist. |
I think I forgot the incantation... Of course, the real problem is that I forgot to set the label... |
@meeseeksdev backport to v6.1.x |
…semantics) in a coordinates test
Arrgghh, just forgot a "v". But now it is done at least. |
…385-on-v6.1.x Backport PR #16385 on branch v6.1.x (TST: fix compatibility with numpy 2 (copy semantics) in a coordinates test)
Thank you ! |
Description
Follow up to #16166
I missed this test in my previous PRs because I didn't have jplephem installed. This was caught in today's daily cron.