DiamondLightSource / DiamondLightSource/dodal

Smargon combined moves should perform omega moves separately

Open
#1,998 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
5
Forks
13
Avg merge
3d 2h
Merged PRs (30d)
10

Description

Following discussions around i03's replacement smargon, it was concluded that the issues around combined moves and slow y-axis caused by the faulty smargon notwithstanding, there are likely to still be issues even with correctly performing smargon.

These are thought to possibly be caused by a) the higher omega speed limit impacting timing calculations for the move and b) real vs virtual axis causing omega moves to make additional smaller moves on the other axes which might already be at their speed limit.

Therefore the plan is even when the new smargon is in place we should perform the omega move first.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.