From 285a0b8f550eae4c78c0a9f4400baa4568d0590d Mon Sep 17 00:00:00 2001 From: Chris Havlin Date: Mon, 3 Jul 2023 14:18:20 -0500 Subject: [PATCH] widg_id always goes up --- src/yt_napari/_widget_reader.py | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/src/yt_napari/_widget_reader.py b/src/yt_napari/_widget_reader.py index 023ca9a..2c6857e 100644 --- a/src/yt_napari/_widget_reader.py +++ b/src/yt_napari/_widget_reader.py @@ -38,6 +38,7 @@ def __init__(self, napari_viewer: "napari.viewer.Viewer", parent=None): self.active_selections = {} self.active_selection_types = {} self.active_selection_layout = QVBoxLayout() + self.widg_id = 0 self.layout().addLayout(self.active_selection_layout) # removing selections @@ -68,13 +69,14 @@ def __init__(self, napari_viewer: "napari.viewer.Viewer", parent=None): def add_a_selection(self): selection_type = self.new_selection_type.currentText() - widg_id = len(self.active_selections) + 1 - widget_name = f"Selection {widg_id}, {selection_type}" - widg_key = f"{selection_type}_{widg_id}" + new_widg_id = self.widg_id + 1 + self.widg_id = new_widg_id + widget_name = f"Selection {new_widg_id}, {selection_type}" + widg_key = f"{selection_type}_{new_widg_id}" new_selection_widget = SelectionEntry(widget_name, selection_type) self.active_selections[widg_key] = new_selection_widget self.active_selection_layout.addWidget(self.active_selections[widg_key]) - self.active_sel_list.insertItem(widg_id - 1, widg_key.replace("_", " ")) + self.active_sel_list.insertItem(new_widg_id - 1, widg_key.replace("_", " ")) # the active_selection_types mapping lists dont need to be cleared self.active_selection_types[widg_key] = selection_type @@ -150,10 +152,7 @@ def load_data(self): data = self._post_load_function(data) # set the metadata - - # NEED TO FIX THIS LOG take_log = new_layer[1]["metadata"]["_is_log"] - # rebuild the metadata dict with reference layer info md = _model_ingestor.create_metadata_dict( data, layer_domain, take_log, reference_layer=ref_layer