-
Notifications
You must be signed in to change notification settings - Fork 83
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
Lockless multi-threaded reads #224
Conversation
Here's a link for the new notebook: https://nbviewer.jupyter.org/github/corteva/rioxarray/blob/73ea5e24d968202f48ca8812f5c00104f9139638/docs/examples/read-locks.ipynb |
Codecov Report
@@ Coverage Diff @@
## master #224 +/- ##
==========================================
+ Coverage 92.13% 92.15% +0.01%
==========================================
Files 11 11
Lines 1259 1287 +28
==========================================
+ Hits 1160 1186 +26
- Misses 99 101 +2
Continue to review full report at Codecov.
|
This is really nice 👍. I like that there are different options available for the file manager. |
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.
Looks good, just a couple text changes.
Whoops, thanks. Fixed those typos. |
@TomAugspurger would you like to squash your commits or do you mind if I do a squash and merge? |
Squash and merge is fine by me. |
Thanks @TomAugspurger 👍 |
Thanks! |
docs/history.rst
for all changes anddocs/rioxarray.rst
for new API