-
Notifications
You must be signed in to change notification settings - Fork 28
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
Load .mrc
using Distributed and with Direct Electron Metadata
#162
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #162 +/- ##
==========================================
+ Coverage 86.02% 86.12% +0.10%
==========================================
Files 81 82 +1
Lines 10387 10471 +84
Branches 2253 2274 +21
==========================================
+ Hits 8935 9018 +83
Misses 931 931
- Partials 521 522 +1 ☔ View full report in Codecov by Sentry. |
c379dc4
to
dc6e0db
Compare
pre-commit.ci autofix |
d9b37a7
to
6cf2307
Compare
…ing and the `np.memmap` function.
for more information, see https://pre-commit.ci
ed69998
to
bde771a
Compare
bde771a
to
1773b10
Compare
1773b10
to
5000af1
Compare
@ericpre so it seems like the Packaging CI seems to fail when adding a new file. Is that correct? It seems like it is trying to pull the file from the registered list to package (and then subsequently prune). Is this something worth trying to fix? Or just wait until the file is merged into main. |
Yes, the packaging failure is normal, it will pass once the PR is merged. |
Description of the change
This adds support for loading .mrc files using the
dask-distributed
backend and memory mapping. This is helpful for loading large files using multiple threads split across many machines.The methods in the
_distributed
file should be fairly extendable and can be applied to other binary files which use thenp.memmap
function.Changes:
memmap_distributed
functiondistributed
andchunks
kwargs to the .mrc file loaderProgress of the PR
memmap_distributed
functionupcoming_changes
folder (seeupcoming_changes/README.rst
),docs/readthedocs.org:rosettasciio
build of this PR (link in github checks)Minimal example of the bug fix or the new feature