Skip to content

Conversation

shoyer
Copy link
Member

@shoyer shoyer commented Sep 28, 2025

This error is only triggered by Zarr 3.1.3 (or newer), which uses os.replace for atomic writes (zarr-developers/zarr-python#3412).

I'm not sure, but I'm not really worried about tracking this down given that tests pass with Zarr 3.1.3 on Python 3.13.

@shoyer shoyer changed the title Silence error in test_zarr_region_chunk_partial_offset on Windows with Python 3.11 Silence error in test_zarr_region_chunk_partial_offset on Windows Sep 28, 2025
@shoyer
Copy link
Member Author

shoyer commented Sep 28, 2025

These PermissionsError usually indicate that we still have a file handle open. I don't see where to fix that here, though.

@shoyer shoyer enabled auto-merge (squash) September 28, 2025 02:43
@shoyer shoyer merged commit b98add1 into pydata:main Sep 28, 2025
36 checks passed
@shoyer shoyer deleted the win-fail branch September 28, 2025 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant