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

Prepare for robot load #1281

Merged
merged 3 commits into from
Mar 27, 2024
Merged

Prepare for robot load #1281

merged 3 commits into from
Mar 27, 2024

Conversation

DominicOram
Copy link
Collaborator

Fixes #1265

To test:

  1. Confirm new unit test passes


yield from bps.mv(composite.smargon.stub_offsets, StubPosition.RESET_TO_ROBOT_LOAD)

# fmt: off
Copy link
Contributor

Choose a reason for hiding this comment

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

What is fmt here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Turns black off. If you let black reformat the line it becomes less readable (I think) as it's not obvious which value corresponds to which axis.

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.

Nice!

@DominicOram DominicOram merged commit 6c251a6 into main Mar 27, 2024
26 checks passed
@DominicOram DominicOram deleted the 1265_rebase_prepare_for_robot_load branch March 27, 2024 09:53
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this pull request Aug 23, 2024
…Source/1265_rebase_prepare_for_robot_load

Prepare for robot load
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.

Prepare for robot load
2 participants