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

Panda grid scans don't correctly check for gridscan_valid #502

Open
olliesilvester opened this issue Sep 17, 2024 · 0 comments · May be fixed by #503
Open

Panda grid scans don't correctly check for gridscan_valid #502

olliesilvester opened this issue Sep 17, 2024 · 0 comments · May be fixed by #503

Comments

@olliesilvester
Copy link
Contributor

olliesilvester commented Sep 17, 2024

PandA grid scans tried to kickoff a gridscan while the BL03I-MO-SGON-01:PGS:SCAN_INVALID was set to invalid (0). IOC logs show that panda grid scan parameters had been updated, so looking at the code, it is unclear why this PV wasn't being checked properly in wait_for_gridscan_valid. See graylog error: https://graylog.diamond.ac.uk/messages/graylog-daq-mx-hyperion_21/3c7fd880-702f-11ef-bc16-26706383e36b

One possible cause:
The invalid PV takes up to 0.1s to update after one of the grid scan parameters change (I think). Hyperion may be doing wait_for_gridscan_valid before the PV has properly updated. Within the time resolution of the IOC logs, this looks possible (but resolution too small to say for sure).
If that is the issue, not sure why we wouldn't have seen it for zebra FGS.

Acceptance Criteria

  • Issue is fixed, potentially by just adding a short sleep in wait_for_gridscan_valid
@olliesilvester olliesilvester linked a pull request Sep 17, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant