European-XFEL / EXtra-geom

Use EuXFEL detector geometry to assemble images
https://extra-geom.rtfd.io
BSD 3-Clause "New" or "Revised" License
5 stars 4 forks source link

Transpose default detector axes #289

Closed egorsobolev closed 5 months ago

egorsobolev commented 5 months ago

This fixes the order of default motor axis for AGIPD. The motor device ids follows the notations of {detector_id}/MOTOR/Q{q}M{m}, where M1 corresponds to the vertical axis and M2 - to the horizontal. It seems natural to keep the same order in the motor position arrays.

takluyver commented 5 months ago

LGTM - I haven't checked the details of which motor does what, but I trust you on that.

egorsobolev commented 5 months ago

Thanks, @takluyver ! I hope I didn't mix anything up this time.