search

BCDA-APS / mdaviz

Data visualization for mda
https://bcda-aps.github.io/mdaviz/
Other
3 stars 0 forks source link

94 check detector selection exist #96

Closed rodolakis closed 7 months ago

rodolakis commented 7 months ago
MDecarabas commented 7 months ago

Traceback (most recent call last): File "/Users/ecodrea/bluesky/mdaviz/mdaviz/select_fields_table_model.py", line 173, in setData self.setCheckbox(index, value) File "/Users/ecodrea/bluesky/mdaviz/mdaviz/select_fields_table_model.py", line 204, in setCheckbox self.checkboxStateChanged.emit(self.plotFields()[0]) ^^^^^^^^^^^^^^^^^ File "/Users/ecodrea/bluesky/mdaviz/mdaviz/select_fields_table_model.py", line 379, in plotFields choices[column_name].append(row)


KeyError: 'Norm'

Lots of errors if you change the variables in the second table
rodolakis commented 7 months ago

Traceback (most recent call last): File "/Users/ecodrea/bluesky/mdaviz/mdaviz/select_fields_table_model.py", line 173, in setData self.setCheckbox(index, value) File "/Users/ecodrea/bluesky/mdaviz/mdaviz/select_fields_table_model.py", line 204, in setCheckbox self.checkboxStateChanged.emit(self.plotFields()[0]) ^^^^^^^^^^^^^^^^^ File "/Users/ecodrea/bluesky/mdaviz/mdaviz/select_fields_table_model.py", line 379, in plotFields choices[column_name].append(row) ~~~^^^^^^^^^^^^^ KeyError: 'Norm'

Lots of errors if you change the variables in the second table

I have not implemented Norm nor Mon, so yeah if you click on any of those it is going to crash. I should probably either hide them for now or make sure I handle that case to prevent crashing. Created issue #97