Skip to content

Adds a new page for Jungfrau rotations#88

Merged
noemifrisina merged 24 commits intomainfrom
59_jf-rotation
Jan 30, 2026
Merged

Adds a new page for Jungfrau rotations#88
noemifrisina merged 24 commits intomainfrom
59_jf-rotation

Conversation

@noemifrisina
Copy link
Contributor

@noemifrisina noemifrisina commented Nov 3, 2025

Closes #59

Needs mx-bluesky#1565

@noemifrisina noemifrisina marked this pull request as ready for review January 21, 2026 16:54
@noemifrisina noemifrisina requested a review from a team as a code owner January 21, 2026 16:54
@olliesilvester olliesilvester self-assigned this Jan 22, 2026
Copy link
Contributor

@olliesilvester olliesilvester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know enough to comment too much on the code quality but it works and matches the parameters needed on the bluesky side

| "ForceSwitchG2"
| "FixG1"
| "FixG2"
| "FixG0";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the ophyd-async Enum I put a comment before the FIX_G0: # Use with caution - this may damage the detector. (I got this from an operations guide for the Jungfrau which is now deprecated).

Could put a similar message on the screen around this option

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will add to the tooltip

@noemifrisina noemifrisina merged commit 5616f83 into main Jan 30, 2026
10 checks passed
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.

Create a new tab for Jungfrau rotation scans

2 participants

Comments