|
"""SiriusPVTimeSerie Class.""" |
@anacso17 , while going through siriuspy various modules for clean up code, I noticed that SiriusPVTimeSerie does not implement PV-type functionality, as all the other modules in siriuspy.epics subpackage do. It merely uses one of its attributes of type PV to implement its methods. None of its methods implements PV methods. The class is like any other class that connects to PVs...
I think this module does not belong to this subpackage. What do you think? It looks like it could be implemented as a Device class...
dev-packages/siriuspy/siriuspy/epics/pv_time_serie.py
Line 1 in adbe62d
@anacso17 , while going through siriuspy various modules for clean up code, I noticed that SiriusPVTimeSerie does not implement PV-type functionality, as all the other modules in siriuspy.epics subpackage do. It merely uses one of its attributes of type PV to implement its methods. None of its methods implements PV methods. The class is like any other class that connects to PVs...
I think this module does not belong to this subpackage. What do you think? It looks like it could be implemented as a Device class...