You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
FAILED astropy\io\fits\tests\test_hdulist.py::TestHDUListFunctions::test_update_resized_header[] - UserWarning: Memory map object was closed but appears to still be referenced. Further access will result in undefined behavior (possibly including segmentation faults).
FAILED astropy\io\fits\tests\test_hdulist.py::TestHDUListFunctions::test_update_resized_header[home_is_data] - UserWarning: Memory map object was closed but appears to still be referenced. Further access will result in undefined behavior (possibly including segmentation faults).
FAILED astropy\io\fits\tests\test_hdulist.py::TestHDUListFunctions::test_update_resized_header[home_is_data, pathlib] - UserWarning: Memory map object was closed but appears to still be referenced. Further access will result in undefined behavior (possibly including segmentation faults).
FAILED astropy\io\fits\tests\test_image.py::TestImageFunctions::test_io_manipulation - UserWarning: Memory map object was closed but appears to still be referenced. Further access will result in undefined behavior (possibly including segmentation faults).
I want to blame Crowdstrike but probably not caused by them.
The warning that we're seeing was introduced in #16581+#16608 but I assumed (at the time) that we were already testing the combo (Windows+numpy2). Evidently we were not. What I don't understand off-hand is why this changed so suddenly.
I'll see if I can adjust this.
- name: Python 3.11 with all optional dependencies (Windows)
windows: py311-test-alldeps
posargs: --durations=50
Maybe one of the "optional dependencies" finally unpinned numpy<2. That is the only logical reason I can think of. We did patch a conda-forge recipe for Windows yesterday but that should not affect CI.
Not sure that makes sense since the only other dependency that changed in the interval (importlib_metadata) has nothing to do with numpy. That said I don't fancy figuring out what happened 🤷♂️
Activity
neutrinoceros commentedon Jul 24, 2024
Here's one of the last runs to succeed:
https://github.com/astropy/astropy/actions/runs/10055868741/job/27793494345
And one of the first to fail:
https://github.com/astropy/astropy/actions/runs/10061840724/job/27812887553
And a diff of their respective resolved envs:
The warning that we're seeing was introduced in #16581+#16608 but I assumed (at the time) that we were already testing the combo (Windows+numpy2). Evidently we were not. What I don't understand off-hand is why this changed so suddenly.
I'll see if I can adjust this.
pllim commentedon Jul 24, 2024
astropy/.github/workflows/ci_workflows.yml
Lines 84 to 86 in d8e98c2
Maybe one of the "optional dependencies" finally unpinned
numpy<2. That is the only logical reason I can think of. We did patch a conda-forge recipe for Windows yesterday but that should not affect CI.neutrinoceros commentedon Jul 24, 2024
Not sure that makes sense since the only other dependency that changed in the interval (importlib_metadata) has nothing to do with numpy. That said I don't fancy figuring out what happened 🤷♂️
pllim commentedon Jul 24, 2024
Could very well be something on server side but yes, let's move on. 😅