Skip to content

Draft: ICSHWI-17614: Initial commit

Carlos Neto requested to merge dev into main

todo list:

  • Check if beam position calculation can/should be done on EPICS level instead of (the current) plc level (most probably depends on the requirements for them)
    • Do we even need thermocouple.plc if we are not calculating delta or error??? Can't we get away with a low T_SMP_MS
  • Should we disable puts on some PVs? Which ones? Or maybe set ASG? Or maybe do all calculations on plc level? Or disable record processing?
  • Think of a better name to ${P}Axis and ${P}Axis-RBV
  • Check if 0 = extracted and 1 = inserted or vice-versa
  • Check if any PV need to be restored by autosave (see below).
    • Update position setpoint based on readback on IOC start (better than using autosave)
  • Check .HIGH alarm values for the thermocouples
  • Enable and test timestamps from EVR. We need EVR prefixes first. Check should be .TSE = 0 or TSE = -2 in this case
  • IT IS NOT CLEAR WHAT THE BRAKE BEHAVIOR SHOULD BE WHEN THINGS GO BAD (including no signal from switches) (check what MPS will do here) (is it even possible to apply brake mid travel?).
  • Check .HIGH alarm values for motion timer and counter
  • Will we have some kind of checks/errors/alarms for the beam position?
  • Stylize .db files
  • Check if alarm values for ${P}WaterTemp are really necessary. ${P}WaterTempOK already provides major severity (but maybe it could work as redundancy?)
  • Check ${P}SetMoving-RB vs $(P)Axis-RBV.THST. Maybe remove ${P}SetMoving-RB after tests? Maybe change $(P)Axis-RBV.THST to midway?
  • Check beam coordinate system: is it really x and y?
  • Improve comments of both .plc files.
Edited by Carlos Neto

Merge request reports