Skip to content

Ticket8830 muon beamcorr - #986

Open
LowriJenkins wants to merge 8 commits into
masterfrom
TICKET8830_MUON_BEAMCORR
Open

LowriJenkins wants to merge 8 commits into
masterfrom
TICKET8830_MUON_BEAMCORR

Conversation

@LowriJenkins

Copy link
Copy Markdown
Contributor

Description of work

Implemented Muon Beam Correction IOC

Ticket

Link to Ticket

Acceptance criteria

  • An IOC exists that applies appropriate corrections to beam positions for each of the south side instruments based on the known strength of the other instruments main magnets.
  • Information on this IOC is exposed to users
  • Existing documentation for what is planned is updated to be documentation for this IOC.
  • Testing is scheduled with the relevant muon beam-lines out of cycle to ensure it functions as expected

Unit tests

Tests are present in the IOC-Test-Framework pullrequest.

Code Review

Functional Tests

  • IOC responds correctly in:
    • Devsim mode
    • Recsim mode
    • Real device, if available
  • Supplementary IOCs (..._0n where n>1) run correctly
  • Log files do not report undefined macros (serach for macLib: macro to find instances of macLib: macro [macro name] is undefined...

Final steps

  • Update the IOC submodule in the main EPICS repo. See Git workflow page for details.
  • Reviewer has merged the associated PR for the release notes

@LowriJenkins

Copy link
Copy Markdown
Contributor Author

As well as running the tests, it's worth running through with the IOC not in simulation mode testing it actually talking to appropriate IOCS (e.g. danfysiks or kepcos).

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 this pull request may close these issues.

1 participant