Skip to content

1998 fix smargon combined moves#1692

Open
rtuck99 wants to merge 6 commits into1598_mod_360from
1998_fix_smargon_combined_moves
Open

1998 fix smargon combined moves#1692
rtuck99 wants to merge 6 commits into1598_mod_360from
1998_fix_smargon_combined_moves

Conversation

@rtuck99
Copy link
Copy Markdown
Contributor

@rtuck99 rtuck99 commented Apr 8, 2026

Fixes

Requires:

Link to dodal PR (if required):

This removes omega from CombinedMove because a) omega moves cannot be reliably parallelised with other axes and b) following mod-360 changes, we wish to apply a phase angle and not an absolute angle which is inconsistent with the other axes in CombinedMove
Instead, when we perform a combined move, the plan explicitly moves the omega axis beforehand

Instructions to reviewer on how to test:

  1. Tests pass
  2. Functionality as described

Checks for reviewer

  • Would the PR title make sense to a user on a set of release notes

@rtuck99 rtuck99 added the hyperion Issues for Hyperion, the Bluesky UDC stack label Apr 8, 2026
@rtuck99 rtuck99 changed the base branch from main to 1598_mod_360 April 8, 2026 14:43
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (1598_mod_360@c7b9f40). Learn more about missing BASE report.

Additional details and impacted files
@@               Coverage Diff               @@
##             1598_mod_360    #1692   +/-   ##
===============================================
  Coverage                ?   92.88%           
===============================================
  Files                   ?      155           
  Lines                   ?     8505           
  Branches                ?        0           
===============================================
  Hits                    ?     7900           
  Misses                  ?      605           
  Partials                ?        0           
Components Coverage Δ
i24 SSX 77.35% <0.00%> (?)
hyperion 98.59% <0.00%> (?)
other 98.31% <0.00%> (?)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rtuck99 rtuck99 force-pushed the 1998_fix_smargon_combined_moves branch from a29c85a to dfd8eab Compare April 9, 2026 09:17
@rtuck99 rtuck99 marked this pull request as ready for review April 9, 2026 09:33
@rtuck99 rtuck99 requested a review from a team as a code owner April 9, 2026 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hyperion Issues for Hyperion, the Bluesky UDC stack

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant