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

eiger2crysalis.py does not support Eiger master files shipped by SPring8 #596

Closed
kif opened this issue Oct 9, 2024 · 2 comments · Fixed by #597
Closed

eiger2crysalis.py does not support Eiger master files shipped by SPring8 #596

kif opened this issue Oct 9, 2024 · 2 comments · Fixed by #597
Assignees
Labels

Comments

@kif
Copy link
Member

kif commented Oct 9, 2024

Reported by a PhD researcher at U. Tokyo

@khsacc
Copy link

khsacc commented Oct 10, 2024

Hi, thank you for dealing with the problem with SP8 data.
I had errors, but the output Esperanto files seem OK. Probably this is because of local data processing at SPring-8: pixel_mask is removed from the master file to reduce data size, cf. https://github.com/keitaroyam/yamtbx/blob/master/doc/eiger-en.md#modification-on-masterh5-at-spring-8
I guess it is ok to ignore this, what do you think?

eiger2crysalis multi_100MGy_4_master.h5
Traceback (most recent call last):■■■■■■■■■■■■■■■ ] 98% Exporting mask as CCD/SET file
File "/Users/hiroki/.pyenv/versions/3.10.3/bin/eiger2crysalis", line 8, in
sys.exit(main())
File "/Users/hiroki/.pyenv/versions/3.10.3/lib/python3.10/site-packages/fabio/app/eiger2crysalis.py", line 619, in main
converter.treat_mask(full=args.calc_mask)
File "/Users/hiroki/.pyenv/versions/3.10.3/lib/python3.10/site-packages/fabio/app/eiger2crysalis.py", line 473, in treat_mask
dstart = self.headers"dom_s"
TypeError: 'float' object is not callable

@kif
Copy link
Member Author

kif commented Oct 10, 2024

Normal "eiger-master" file contain not information about the goniometer:
You need to specify which axis rotate and at which speed, for example "--phi=2*index"

@kif kif closed this as completed in #597 Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants