Add delmic load function #139
Conversation
|
Tests are failing, because |
|
Hi Jonas,
Yes I'll do it later today, sorry about that.
Best,
Noémie
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #139 +/- ##
==========================================
+ Coverage 85.24% 85.68% +0.43%
==========================================
Files 73 76 +3
Lines 9034 9841 +807
Branches 2045 2161 +116
==========================================
+ Hits 7701 8432 +731
- Misses 872 909 +37
- Partials 461 500 +39
☔ View full report in Codecov by Sentry. |
…remove issue with reading h5 and hdf5 fomats
|
Thanks for putting this forward @noemiebonnet As far as I understand, it only reads spectral images and not other types of Delmic-HDF5 files (e.g. single spectra, intensity maps, angle-resolved data) so far? Note that we changed the placement and handling of the test data files in #123 Concerning the metadata, it would be great if the full metadata tree from the hdf5 file could be placed under For the fields mentioned in https://docs.lumispy.org/en/latest/user_guide/metadata_structure.html it would be great if they can be mapped to the |
| @@ -0,0 +1,8 @@ | |||
| name: delmic | |||
There was a problem hiding this comment.
| name: delmic | |
| name: Delmic | |
| }, | ||
| ] | ||
|
|
||
| metadata = {"signal": {"signal_type": "", "quantity": "Intensity (counts)"}} |
There was a problem hiding this comment.
If LumiSpy is installed, signal_type should be set to CL to provide the CL specific functions.
|
|
||
| metadata = {"signal": {"signal_type": "", "quantity": "Intensity (counts)"}} | ||
|
|
||
| original_metadata = dict(DimensionScaleX="182", DimensionScaleY="132") |
There was a problem hiding this comment.
I guess these fixed values are just from testing so far?
|
I added a progress tracker to the initial comment. It helps show others how far this PR is, feel free to amend the list by editing the comment. |
Description of the change
Progress of the PR
upcoming_changesfolder (seeupcoming_changes/README.rst),readthedocsdoc build of this PR (link in github checks)