-
Notifications
You must be signed in to change notification settings - Fork 5
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
VMA Fixes #17
VMA Fixes #17
Conversation
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.
Also small suggestion (can't put a comment there), can you change the @lru_cache(65536)
at line 94 to a self._extent = lru_cache(65536)(self._extent)
at the bottom of __init__
?
Done! Could you please explain your reasons for asking this? We don't see how this benefits the code, but we suspect you've got good reasons for doing so 😄 |
https://rednafi.github.io/python/lru_cache_on_methods/ TL;DR: |
Co-authored-by: Erik Schamper <1254028+Schamper@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## main #17 +/- ##
==========================================
+ Coverage 60.96% 60.99% +0.02%
==========================================
Files 29 29
Lines 2375 2374 -1
==========================================
Hits 1448 1448
+ Misses 927 926 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Two bugs related to VMA are covered in this PR:
_iter_clusters
as well, in order to return the rightblock_offset
.