ISISComputingGroup / IBEX

Top level repository for IBEX stories
5 stars 2 forks source link

RIKEN: Custom Magnet PSU ("synoptic") OPI #5872

Closed kjwoodsISIS closed 1 year ago

kjwoodsISIS commented 4 years ago

As a muon scientist I would like an overview showing the status of all the magnet PSUs in the RIKEN beamline in the form of a 4-branch beamline synoptic. The overview should also display the status of the separator HV PSUs. The status information is provided by a PLC.

Acceptance Criteria

  1. I can see an overview of all the RIKEN front-end PSUs in a single view.
  2. For each PSU, the overview should display an icon labelled with the PSU name and indicating:
    • the power status (ON/OFF)
    • the alarm status (In Alarm/Not in Alarm)
    • (optional - if space allows) the interlock status
  3. For each PSU, the icon should function as button
    • on clicking the button, I can drill-down to the individual PSU OPI

Notes

  1. The PSU status information is provided by a PLC
    • the PLC is a Schneider M580 (Modbus)
    • the PLC is physically located in RIKEN control room
  2. This OPI is intended to be a "PSU synoptic". The muon scientists would like it to closely resemble the actual beamline branching structure.
    • c.f. the existing RIKEN front-end synoptic
      • which is topologically correct, if not entirely realistic
      • maybe a fully realistic OPI is not feasible (e.g. insufficient space on the screen)
  3. Consult the photograph of the PSUs HMI screen when creating the OPI.
  4. Some ideas on what a synoptic could look like are sketched in the RIKEN Handover slides.
daryakoskeroglu commented 2 years ago

Created 2 versions of RIKEN beamline- PSUs/Magnets OPI.

My references were from slide 22 and slide 23 on RIKEN Handover presentation which can be found here : http://www.facilities.rl.ac.uk/isis/computing/ICPdiscussions/Forms/AllItems.aspx?RootFolder=%2Fisis%2Fcomputing%2FICPdiscussions%2FRIKEN%20FE&FolderCTID=0x01200027AD8F05966A2748B3B04C98BB5B442B&View={F2C33C51-70E6-4343-B937-2C59A2568306}

PVs are currently not connected and more information needed regarding the colours of magnets/PSUs shown on the OPI.

The changes are pushed to branch "Ticket5872_custom_magnet_psu_opi".

daryakoskeroglu commented 2 years ago

FYI for the handover I put the design slides (design process) in here, "RIKEN Magnet PSU OPI": http://www.facilities.rl.ac.uk/isis/computing/ICPdiscussions/Forms/AllItems.aspx?RootFolder=%2Fisis%2Fcomputing%2FICPdiscussions%2FRIKEN%20FE&FolderCTID=0x01200027AD8F05966A2748B3B04C98BB5B442B&View={F2C33C51-70E6-4343-B937-2C59A2568306}

Things to do:

JackEAllen commented 2 years ago

Handover Notes

These handover notes are for when we have IOC's to populate the OPI with. All files are placed inside the folder ibex_gui\base\uk.ac.stfc.isis.ibex.opis\resources\RIKEN\ on the branch Ticket5872_custom_magnet_psu_opi

RIKEN_magnet_PSUs.opi

Populate the correct IOCs for the splitter magnets, kickers, solenoid, SEP (N & P), XFD and beamline settings: OPI Attribute Image
Splitter magnet splitter magnet
Splitter magnet splitter_magnet_2
Kicker kicker
Solenoid solenoid
Beamline settings beamline_settings

Populate the IOCs for each PSU which have currently been hard coded using template values into the macro for the linking container. OPI Attribute Image
PSU PSU
Macros PSU_macro_table

Populate the IOC's required for each port button through the action tab for each button to be passed to the individual port tables. 4 IOC Default hard coded values have been set for you to alter: button>Actions>Macros>Value

OPI Attribute Image
Port Button port_button
Set Actions set_actions
Edit Macros edit_macros_table

indiviudal_port_table_port.opi

For each port table individual_port_table_port<N>.opi add or remove any row not required based on the number of IOCs for the given port.

OPI Attribute Image
Port Table port_table

The macros passed through the port button from the main OPI should now populate the port table.

JackEAllen commented 2 years ago

Placing in impeded until we have IOCs to populate the OPI with and test.

LowriJenkins commented 1 year ago

This Ticket is based off of https://github.com/ISISComputingGroup/IBEX/issues/7241 and should not be merged until that has been. For testing, ask david for the components needed for this, run the iocs in record simulation, for testing individual parts of the OPI change:

rerpha commented 1 year ago

This Ticket is based off of https://github.com/ISISComputingGroup/IBEX/issues/7241 and should not be merged until that has been.

should it be in impeded then?

LowriJenkins commented 1 year ago

This Ticket is based off of #7241 and should not be merged until that has been.

should it be in impeded then?

its a fairly big ticket, and the changes can still be reviewed since it contains that branches commits, I thought that given its a fairly large ticket #7241 might be finished and reviewed while this is still under-review, but i'm happy to move this into impeded instead if you think thats the right move.