moxa iologik: Avoid covering displayed values with disconnection errrors in OPI

[FreddieAkeroyd]

Description of work

show unaliased values

To test

check all values show in OPI

Acceptance criteria

List the acceptance criteria for the PR

• ** If this PR changes GALIL / GALILMUL ioc are these changes applicable to both the old (galil-old branch based) and new (master branch based) drivers? If so have all appropriate PRs been created **

---

Code Review

• A copy of the manual has been placed on the shared drive
• Pertitent information 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?
• Do the IOC and support module conform to the new build guidelines
• Have the changes been recorded appropriately in a PR for release notes?
• Is the device's flow control setting correct? For most devices flow control should be OFF.

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

[Tom-Willemsen]

Looking at the original PRs, this functionality (hiding unaliased PVs) was explicitly included. I guess it was so that people didn't see lots of irrelevant/confusing inputs in the OPI if they were only using one or two of them.

I'm not convinced we want this change?

If we do want this change, we need:

• A corresponding change in the OPI side to remove the now-redundant rules/scripts.
• Release notes as this is user facing

[FreddieAkeroyd]

What i was trying to avoiding was the rather horrible mess of disconnected boxes that seemed to get left when aliases were not added, i think the STAT pv used by the OPI only existed if the PV was aliased. Maybe we can show a value greyed out if it is unaliased? Of if we prefer to hide them, keep db changes other than the ones for moxa12XX_aliases.db so at least we get the PVs the OPI is expecting defined at all times even if not displayed

[Tom-Willemsen]

Maybe we can show a value greyed out if it is unaliased? Of if we prefer to hide them, keep db changes other than the ones for moxa12XX_aliases.db so at least we get the PVs the OPI is expecting defined at all times even if not displayed

That sounds reasonable to me.

[FreddieAkeroyd]

The greying out would need an OPI change, the other probably not. At this stage should we just make the PVs available but hidden, which gets round one problem where the disconnected errors actually blocked out what you had tried to make visible by aliases