DiamondLightSource / DiamondLightSource/pmac

Populate all of the PMAC_CONTROLER:AXIS:N PVs

Open
#139 0 comments 0 reactions 1 assignee Claimed by @LeandroMartinsdS View on GitHub
Dominant language
C++
Stars
31
Forks
25
PR merge metrics
No merged PRs in 30d

Description

At present each instantiation of `dls_pmac_asyn_motor.template` or `` also adds an alias that has a deterministic, numbered PV.

see:
https://github.com/DiamondLightSource/pmac/blob/afe81f8bb9179c3a20eff351f30bc6cfd1539ad9/pmacApp/Db/dls_pmac_asyn_motor.template#L69-L73
https://github.com/DiamondLightSource/pmac/blob/afe81f8bb9179c3a20eff351f30bc6cfd1539ad9/pmacApp/Db/dls_pmac_asyn_motor.template#L14-L30

The problem with this is that the screen pmacAxes.edl will often have gaps in the PVs it tries to render. This causes edm to search for non-existent PVS. This is also a problem for Ophyd objects trying to drive pmacs.

The proposed fix is to instantiate some minimal records for all 16 axes with no DTYP (defaulting to softrecord). Then the aliases would override those records for which we do need a connection to an axes and the remaining ones would be left as minimal placeholders that fix the issue described above.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.