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

Kepco BOP PSU: Investigate communications issue seen on Muon Zerofield system [Timebox: 2 days] #7778

Closed
2 tasks
davidkeymer opened this issue May 10, 2023 · 6 comments
Assignees
Labels

Comments

@davidkeymer
Copy link
Contributor

davidkeymer commented May 10, 2023

As a muon (or neutron) instrument scientist, I would like the communications with my Kepco BOP PSU to be reliable.

While investigating recent zerofield issues seen on MuSR, it was discovered that the PSU occasionally sent (or a reply was misinterpreted) an invalid setpoint. e.g. actual setpoint: 0.74, readback: 9.74.

Acceptance criteria

  • Source of communications problem is found
  • Any resultant work is split out into further tickets

Notes

@KathrynBaker
Copy link
Member

From Ticket #7719

As a developer I would like to remove ExtraInput = ignore from the kepco protocol https://github.com/ISISComputingGroup/EPICS-kepco/blob/master/kepcoApp/protocol/kepco.proto as it potentially increases the chance of a corrupt reply being interpreted as a good one.

We may also want to consider adding ;*OPC? to the end of set commands which should return 1 when it completes, however i might have already tries this and found our cards did not support it

@KathrynBaker KathrynBaker changed the title Kepco BOP PSU: Investigate communications issue seen on Muon Zerofield system Kepco BOP PSU: Investigate communications issue seen on Muon Zerofield system [Timebox: 2 days] May 11, 2023
@KathrynBaker KathrynBaker added the 8 label May 11, 2023
@github-actions github-actions bot added ready and removed proposal labels May 11, 2023
@KathrynBaker KathrynBaker added this to the SPRINT_2023_05_11 milestone May 11, 2023
@KathrynBaker KathrynBaker removed this from the SPRINT_2023_05_11 milestone Jun 7, 2023
@KathrynBaker KathrynBaker added this to the SPRINT_2023_06_08 milestone Jun 8, 2023
@davidkeymer davidkeymer self-assigned this Jun 29, 2023
@github-actions github-actions bot added impeded and removed ready labels Jul 12, 2023
@davidkeymer
Copy link
Contributor Author

Impeded until can get an opportunity to test the suggested changes to the Kepco protocol file.

@FreddieAkeroyd
Copy link
Member

One of the kepcos giving issue was right on the full/quarter range point - as additional accuracy not needed have added #7945 as this continued switching may cause a delay?

@rerpha
Copy link
Contributor

rerpha commented May 13, 2024

has this been patched onto MuSR? if so, it should be added here https://github.com/ISISComputingGroup/IBEX/wiki

@KathrynBaker KathrynBaker moved this from In Progress to Review in PI_2024_02 May 13, 2024
@rerpha rerpha closed this as completed May 28, 2024
@github-project-automation github-project-automation bot moved this from Review to Done in PI_2024_02 May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: In Progress
Status: Done
Development

No branches or pull requests

4 participants