Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

Changes to support ophyd_async for mirror voltages #1453

Merged
merged 3 commits into from
Jul 11, 2024

Conversation

rtuck99
Copy link
Contributor

@rtuck99 rtuck99 commented Jun 18, 2024

These changes fix breakages caused by updating the VFMMirrorVoltages device to use ophyd async

Fixes DiamondLightSource/dodal#604

Link to dodal PR (if required): DiamondLightSource/dodal#636
(remember to update setup.cfg with the dodal commit tag if you need it for tests to pass!)

To test:

  1. Tests still pass
  2. mirror stripe change continues to work

@rtuck99 rtuck99 marked this pull request as ready for review June 18, 2024 14:46
@rtuck99 rtuck99 requested review from d-perl and DominicOram June 18, 2024 14:46
Copy link
Collaborator

@DominicOram DominicOram left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@DominicOram DominicOram changed the title (DiamondLightSource/dodal#604) changes to support ophyd_async for mirror voltages Changes to support ophyd_async for mirror voltages Jul 11, 2024
@DominicOram DominicOram merged commit c995b94 into main Jul 11, 2024
4 checks passed
@DominicOram DominicOram deleted the dodal_604_ophyd_async_device_for_mirror_voltages branch July 11, 2024 16:59
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this pull request Aug 23, 2024
…e/hyperion#1453)

* (DiamondLightSource/dodalDiamondLightSource/hyperion#604) changes to support ophyd_async mirror voltages, use of DeviceVector therein

---------

Co-authored-by: Dominic Oram <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ophyd-async device for mirror voltages
2 participants