Skip to content

bug fix combined-lock-locked issue #10843#11022

Merged
max-sixty merged 1 commit intopydata:mainfrom
samay2504:fix/xarray-10843-combined-lock-locked
Jan 4, 2026
Merged

bug fix combined-lock-locked issue #10843#11022
max-sixty merged 1 commit intopydata:mainfrom
samay2504:fix/xarray-10843-combined-lock-locked

Conversation

@samay2504
Copy link
Contributor

No description provided.

@max-sixty
Copy link
Collaborator

ref #10843

@@ -0,0 +1,78 @@
"""Test for issue #10843: CombinedLock.locked() always returns true."""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we have this as a test in a more idomatic place? ty!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@samay2504 samay2504 force-pushed the fix/xarray-10843-combined-lock-locked branch from ab0333a to 6f7885f Compare December 16, 2025 11:50
@garg-khushi
Copy link

Verified locally.
Checked out this branch and ran the backend lock tests:
python -m pytest -o addopts= xarray/tests/test_backends_locks.py
All tests pass. The change to call lock.locked() instead of accessing it as a property works as expected on my machine.

@max-sixty max-sixty merged commit 018ad08 into pydata:main Jan 4, 2026
40 checks passed
@welcome
Copy link

welcome bot commented Jan 4, 2026

Congratulations on completing your first pull request! Welcome to Xarray! We are proud of you, and hope to see you again! celebration gif

dcherian added a commit to dcherian/xarray that referenced this pull request Jan 28, 2026
- Add missing bug fixes: pydata#11064, pydata#11026, pydata#11022, pydata#11005, pydata#10980, pydata#10788, pydata#11085
- Add missing documentation entries: pydata#10958, pydata#11094, pydata#11080
- Add missing performance entry: pydata#11105
- Add Internal Changes section with pydata#11108
- Fix rST roles: use :py:func:, :py:meth:, :py:class: consistently
- Fix missing commas between :issue: and :pull: references
- Fix typo in Dataset.identical entry
- Fix wrong PR number for coord ordering (11098 -> 11091)
- Add missing :pull: reference for arithmetic_compat (pydata#10943)

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants