Skip to content
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

Parsing .mat generated by the latest matlab requires h5py, which is very different from using scipy. #167

Open
MichaelEhrlichman opened this issue Jun 10, 2024 · 3 comments

Comments

@MichaelEhrlichman
Copy link
Member

I encountered this bug when looking at
https://github.com/slaclab/lcls-tools/blob/main/examples/matlab_image_analysis.py

The error message is:

Error loading mat file ProfMon-PROF_COL0_535-2024-06-09-175304.mat: Please use HDF reader for matlab v7.3 files, e.g. h5py

Note that using h5py to parse .mat files is very different from using scipy and can be rather awkward.

@MattKing06
Copy link
Collaborator

MattKing06 commented Jun 11, 2024

Hi @MichaelEhrlichman, would it be possible for you to upload the matlab file in question. I've tried testing this example with the datasets we have in the test directory and they must have been created with a different version of matlab as I didn't get the error you mention.

It'd be good to get coverage for files with differing matlab versions.

@MichaelEhrlichman
Copy link
Member Author

OrbitGrid--2023-06-04-152116.mat.gz

@MichaelEhrlichman
Copy link
Member Author

Check out the static/bpmList data type.
Here is a Slack discussion on this topic.
https://slac.slack.com/archives/C9UN7J97B/p1718063303838939

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants