-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix old I03 data not processing #737
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #737 +/- ##
==========================================
- Coverage 42.15% 41.73% -0.42%
==========================================
Files 188 188
Lines 16764 16877 +113
Branches 3196 3232 +36
==========================================
- Hits 7067 7044 -23
- Misses 9052 9178 +126
- Partials 645 655 +10 |
This reverts commit 8a57854.
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.
Approve as-is because would fix the problem as reported, I assume, but looks like there is a gap woth handling foo.nxs
Some data taken with the Eiger pre-2020 was written: - With a 64-bit VDS pointing to 32-bit data arrays, - Without the metafile being linked in the main file, - Without the eiger self-reported data type being in the metafile. Solving these allows DIALS to know that the VDS is wrong, and that it is safe to convert down to 32-bit. Soves dials/dials#2669.
Thanks, confirmed I can process that data set now 👍 |
Some data taken with the Eiger pre-2020 was written: - With a 64-bit VDS pointing to 32-bit data arrays, - Without the metafile being linked in the main file, - Without the eiger self-reported data type being in the metafile. Solving these allows DIALS to know that the VDS is wrong, and that it is safe to convert down to 32-bit. Soves dials/dials#2669.
This data did not process, because:
_meta.h5
file (it was not linked from the.nxs
, which was assumed). There is now a fallback that uses pure filename instead of trying to determine it from the main file._meta.h5
file didn't have a stored "Detector bit depth" field. This now uses the per-imagebitdepth
field.Fixes dials/dials#2669.