Open prjemian opened 2 years ago
While developing better unit tests for the nxdl_manager module (#163), noticed this line: https://github.com/prjemian/punx/blob/f0e707d1545e75b8aa5184334454849f03e24d38/punx/nxdl_manager.py#L527
nxdl_manager
The consequences are that the values in the group's .attributes dictionary are text values directly from the XML rather than instances of nxdl_manager.NXDL__attribute.
.attributes
nxdl_manager.NXDL__attribute
Seems irregular. Why is this? Should it be changed?
Still there: https://github.com/prjemian/punx/blob/af8eefc5b2dc65af7739fb0fee65d917126fbf1e/punx/nxdl_manager.py#L630
prjemian
commented
2 years ago
While developing better unit tests for the
nxdl_managermodule (#163), noticed this line: https://github.com/prjemian/punx/blob/f0e707d1545e75b8aa5184334454849f03e24d38/punx/nxdl_manager.py#L527The consequences are that the values in the group's
.attributesdictionary are text values directly from the XML rather than instances ofnxdl_manager.NXDL__attribute.Seems irregular. Why is this? Should it be changed?