-
-
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
Cythonize checking out of bounds latitudes #16327
base: main
Are you sure you want to change the base?
Conversation
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.
|
The CI failure here looks related:
|
Ah, outside the coordinates package... I only ran the tests of |
The FITS tests are relatively easy, endianness needs to be switched / big endian supported on little endian cpu. However, I am more confused about the uncertainty tests. How could I convinve this object to let me see its data as a simple 1d array:
|
As there is this and also the wrapping cython function, I think we should only have one cython file for astropy.coordinates, because there is quite an overhead in terms of number of lines of code for each Cython file which slows down compilation. |
7897d08
to
b9712c3
Compare
@astrofrog Ok, I'll rebase the second PR to use a single cython file once one is merged |
@maxnoe which one should we prioritize in reviewing ? |
Let's do #16326 first |
Gotcha. Can you make this one a draft then ? |
Description
Essentially the same as #16326 but for the out of limits checking in
Latitude
;Profiling
Latitude(angle)
for different shapes / number of nans in the array before and after:Fixes #