BCDA-APS / gemviz

Data visualization for tiled
https://bcda-aps.github.io/gemviz/
Other
4 stars 0 forks source link

self.selections should follow Qt's get/set pattern #201

Closed prjemian closed 8 months ago

prjemian commented 8 months ago

When self.selections is used: https://github.com/BCDA-APS/gemviz/blob/4ceef1ee93b9d65bdec15c81592493d514529a78/gemviz/select_fields_tablemodel.py#L218

it is not handled with setSelections() and a self._selections internal attribute. Refactor to be consistent with Qt's get/set pattern. We use that everywhere else. Or at least try to.