epics-containers/pvi

Consider refactoring Signal class hierarchy

Closed this issue · 2 comments

GDYendell commented
  • Base Signal inherited by all instead of Component directly
  • Base Signal to have an access_mode to be implemented by child classes
  • SignalRW to inherit from SignalR and SignalW
  • SignalX to inherit from SignalW
  • {Read,Write}Widget -> Widget
  • API to walk all Signals of Tree so that the caller doesn't have to check if the values are of type Signal
GDYendell commented

I don't think it is possible to have SignalR(Signal), SignalW(Signal) and SignalRW(SignalR, SignalW) because SignalRW.pv should actually be SignalRW.read_pv when used as a SignalR...

GDYendell commented

Resolved by #88