Skip to content

APS Sector 21 / LC-CAT Pilatus Support#877

Open
graeme-winter wants to merge 6 commits intomainfrom
aps-s21-omega-pilatus
Open

APS Sector 21 / LC-CAT Pilatus Support#877
graeme-winter wants to merge 6 commits intomainfrom
aps-s21-omega-pilatus

Conversation

@graeme-winter
Copy link
Collaborator

Support Pilatus format for Sector 21 / LS-CAT at the APS.

They use Omega / Omega_increment in place of Start_angle and Angle_increment

graeme-winter and others added 6 commits November 21, 2025 12:06
Make assumptions that the fast depends on the slow which depends on module
offset, and none of these add rotations into the mix (fair, for a DECTRIS
detector)

Resolves #865
In addition to allowing Start_angle / Angle_increment to define the
scan information: this is what is in use at APS S21.
@graeme-winter
Copy link
Collaborator Author

Additional: this instrument used to live at SBC I think so gets caught in

from __future__ import annotations

import sys

from dxtbx.format.FormatCBFMiniPilatus import FormatCBFMiniPilatus


class FormatCBFMiniPilatus3AOS19ID6MSN132(FormatCBFMiniPilatus):
    """A class for reading mini CBF format Pilatus3 images for 6M SN 132 @ APS19ID."""

    @staticmethod
    def understand(image_file):
        """Check to see if this looks like an Pilatus mini CBF format image,
        i.e. we can make sense of it."""

        header = FormatCBFMiniPilatus.get_cbf_header(image_file)

        for record in header.split("\n"):
            if (
                "# Detector" in record
                and "PILATUS3" in record
                and "S/N 60-0132" in header
            ):
                return True

        return False

    def _goniometer(self):
        """19ID has reversed goniometer"""

        return self._goniometer_factory.single_axis_reverse()


if __name__ == "__main__":
    for arg in sys.argv[1:]:
        print(FormatCBFMiniPilatus3AOS19ID6MSN132.understand(arg))

which is no longer correct - I have asked for the timestamp when the switch occurrred

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

Successfully merging this pull request may close these issues.

2 participants