epics-containers / pvi

EPICS PV Interface described in YAML
https://epics-containers.github.io/pvi
Apache License 2.0
4 stars 3 forks source link

Fix SignalRW read_pv convert #93

Closed GDYendell closed 8 months ago

GDYendell commented 8 months ago

The conversion was defaulting to use name + _RBV for read_pv, which is now not correct.

This fix makes SettingPair pass in the read_record so that the pv comes from the template.

It would probably be possible to refactor these classes so that more fields are required and checks for None can be removed.

GDYendell commented 8 months ago

@gilesknap apologies you may need to check the convert you did for motor with this latest version. If you had SignalRWs the read_pvs will be wrong.