Ticket2813 riken changeover logic

[Tom-Willemsen]

Description of work

Adds a coordinator IOC.

This IOC is intended to be reusable in different configurations but initially is being used for two (very similar) tasks:

• Riken port changeover
• Riken RB2 mode change

To test

ISISComputingGroup/IBEX#2813

Acceptance criteria

• Changeovers are started when input PV is changed (currently pointing at SIMPLE)

• Waits for relevant power supplies to be OFF

• Sets DISP field on power setpoints

• Allows changeover (currently pointing at SIMPLE)

• Waits for changeover to be complete

• Re-enables DISP on power PVs

• Process can be safely interrupted at any point in the above process.

---

Code Review

• Pertitent information and a copy of the manual has been stored in the wiki
• Does the IOC conform to IBEX standards?
  • PV naming.
  • Disable records
  • Record simulation
  • Finishing touches
• If an OPI has been modified, does it conform to the style guidelines?
• Have the changes been recorded appropriately in the release notes?

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 moved the release notes entry for this ticket in the "Changes merged into master" section

[DominicOram]

Can you avoid the use of MASTER_RELEASE as per ISISComputingGroup/IBEX#2320. See https://github.com/ISISComputingGroup/EPICS-ioc/blob/master/ISISDAE/configure/RELEASE for an example. Add this IOC to the ticket when done.

[Tom-Willemsen]

done.

[DominicOram]

For readability, only pass the macros which are actually required.

[DominicOram]

In which case do we need to set them as EPICS env variables at all or can we just pass them straight to the seq in the st.cmd?

[Tom-Willemsen]

The reason you can't pass them straight to seq in the individual st.cmds is that starting a sequence program before iocInit is called is a bad idea (undefined behaviour?)

[DominicOram]

Seems to work as described, some small issues:

• Please write a page on the wiki about this IOC, both the things you think could be used in other situations and the bits specific to this case.
• This isn't a requirement of this ticket but would it be useful to add an OPI for this functionality? e.g. turn off all three power supplies with one button, give feedback on what the next steps are. @KathrynBaker?
• As discussed:
  • Can you pull the PLC into a macro?
  • Only one group is needed

[Tom-Willemsen]

Review comments addressed. Documentation added.