Skip to content

Auto shutter#172

Draft
MarkRivers wants to merge 13 commits intomasterfrom
auto_shutter
Draft

Auto shutter#172
MarkRivers wants to merge 13 commits intomasterfrom
auto_shutter

Conversation

@MarkRivers
Copy link
Collaborator

Adds ability to automatically close the shutter at the end of a scan.

@MarkRivers MarkRivers marked this pull request as draft April 17, 2025 22:13
@MarkRivers
Copy link
Collaborator Author

This PR adds a number of things from 13-BM-D.

  • Ability to automatically close the shutter at the end of a scan. This is very helpful in reducing radiation damage to samples and optics.
  • Widgets to manually open and close the shutter on the tomoScan OPI screen.
  • Added support for the Mikrotron and Hamamatsu Orca Fire CoaXPress cameras
  • At the end of the scan sets the HDFLocation back to the projection location. Previously it could be left in the flat location, depending on what was collected last. That meant that simple radiography data collected after a tomoscan would have the projections in the wrong location.
  • Fixed errors in 13bm_mcs that prevented scans in the negative direction from working.
  • Increased the wait time in tomoscan_pso::collect projections from 30 to 90 seconds. 30 was not enough for file writing to complete in some cases. We might want a better method here.
  • Changed tomoscan_pso::program_PSO so the final motor position is 1 more angle step This is needed because the Orca ignores the first PSO pulse. It should not be a problem for other systems, except perhaps for soft limits.

This is a screenshot of the new tomoScan.adl with the new shutter widgets.
image

This PR is draft for now, so it does not get merged until there is time for discussion.

Copy link
Member

@decarlof decarlof left a comment

Choose a reason for hiding this comment

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

@MarkRivers sorry I didn’t see this earlier, I missed the notification. I have reviewed the changes, and they’re excellent improvements that we should propagate to our beamlines as well.

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