Skip to content

Commit

Permalink
fix test failures
Browse files Browse the repository at this point in the history
  • Loading branch information
kecnry committed Dec 13, 2024
1 parent 42392e2 commit 82203ff
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
14 changes: 7 additions & 7 deletions jdaviz/configs/default/plugins/subset_tools/subset_tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -115,8 +115,8 @@ class SubsetTools(PluginTemplateMixin):
icon_radialtocheck = Unicode(read_icon(os.path.join(ICON_DIR, 'radialtocheck.svg'), 'svg+xml')).tag(sync=True) # noqa
icon_checktoradial = Unicode(read_icon(os.path.join(ICON_DIR, 'checktoradial.svg'), 'svg+xml')).tag(sync=True) # noqa

combination_items = List([]).tag(sync=True)
combination_selected = Any().tag(sync=True)
combination_mode_items = List([]).tag(sync=True)
combination_mode_selected = Any().tag(sync=True)

def __init__(self, *args, **kwargs):
super().__init__(*args, **kwargs)
Expand Down Expand Up @@ -159,8 +159,8 @@ def __init__(self, *args, **kwargs):
multiselect=None)

self.combination_mode = SelectPluginComponent(self,
items='combination_items',
selected='combination_selected',
items='combination_mode_items',
selected='combination_mode_selected',
manual_options=COMBO_OPTIONS)

@property
Expand Down Expand Up @@ -1026,11 +1026,11 @@ def _load_regions(self, regions, combination_mode=None, max_num_regions=None,
if return_bad_regions:
return bad_regions

@observe('combination_selected')
def _combination_selected_updated(self, change):
@observe('combination_mode_selected')
def _combination_mode_selected_updated(self, change):
self.app.session.edit_subset_mode.mode = SUBSET_MODES_PRETTY[change['new']]

def _update_combination_mode(self):
if self.app.session.edit_subset_mode.mode in SUBSET_TO_PRETTY.keys():
self.combination_mode.selected = SUBSET_TO_PRETTY[
self.combination_mode_selected = SUBSET_TO_PRETTY[
self.app.session.edit_subset_mode.mode]
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@

<v-row v-if="api_hints_enabled" style="margin-top: -32px">
<span class="api-hint">
plg.combination_mode = '{{ combination_selected }}'
plg.combination_mode = '{{ combination_mode_selected }}'
</span>
</v-row>

Expand Down

0 comments on commit 82203ff

Please sign in to comment.