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

Adjust and merge i22 Pilatus Ophyd Async device #406

Closed
keithralphs opened this issue Apr 2, 2024 · 0 comments · Fixed by #424 or #500
Closed

Adjust and merge i22 Pilatus Ophyd Async device #406

keithralphs opened this issue Apr 2, 2024 · 0 comments · Fixed by #424 or #500
Assignees

Comments

@keithralphs
Copy link
Contributor

keithralphs commented Apr 2, 2024

A Pilatus detector Ophyd Async device exists in a Draft Pull request the DAQ-5039 branch. This needs to be examined and adjusted, if necessary, to be compatible with the 0.3a1 release of Ophyd Async and then merged into main with appropriate tests to confirm its operation (this assumes that the v0.3a1 dependency has already been established in the main branch). Ensure thorough unit tests are present.

Acceptance Criteria

  • Pilatus device available
  • Config for SAXS and WAXS available in i22 beamline module
  • All unit tests for the device pass
  • The device has been tested in a bluesky terminal on i22 using the count plan and performs correctly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants