BCDA-APS / adl2pydm

Convert MEDM's .adl files to PyDM's .ui format
Other
0 stars 4 forks source link

(polygon widget): Point should be output as Python tuple(number, number) #86

Closed prjemian closed 2 years ago

prjemian commented 2 years ago

For compatibility with https://github.com/slaclab/pydm/issues/579#issuecomment-1124035799, polygon points should now be represented as either tuple(number, number) or list(number, number).

prjemian commented 2 years ago

https://github.com/BCDA-APS/adl2pydm/blob/391e20eb001c4a6659a0ae5993cdd2f9e391a502/adl2pydm/adl_parser.py#L258-L266

prjemian commented 2 years ago

https://github.com/BCDA-APS/adl2pydm/blob/1f45e49f560ad249c8319d999101bc8d9f1d4ec8/tests/test_output_handler.py#L671-L680

prjemian commented 2 years ago

Unit tests will fail until a new release of adl2pydm includes the required feature.