Skip to content

Commit

Permalink
Fix Line Analysis behavior before any lines are loaded (spacetelescop…
Browse files Browse the repository at this point in the history
…e#2831)

* Fix behavior before any lines are loaded

* Update changelog
  • Loading branch information
rosteen authored Apr 24, 2024
1 parent b3e32ca commit f667af7
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ Other Changes and Additions

- Bump required Python version to 3.10. [#2757]

- Line menu in Redshift from Centroid section of Line Analysis now shows values in current units. [#2816]
- Line menu in Redshift from Centroid section of Line Analysis now shows values in current units. [#2816, #2831]

3.9.2 (unreleased)
==================
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ class LineAnalysis(PluginTemplateMixin, DatasetSelectMixin, SpectralSubsetSelect
results_computing = Bool(False).tag(sync=True)
results = List().tag(sync=True)
results_centroid = Float().tag(sync=True) # stored in AA units
line_menu_items = List([{}]).tag(sync=True)
line_menu_items = List([]).tag(sync=True)
sync_identify = Bool(True).tag(sync=True)
sync_identify_icon_enabled = Unicode(read_icon(os.path.join(ICON_DIR, 'line_select.svg'), 'svg+xml')).tag(sync=True) # noqa
sync_identify_icon_disabled = Unicode(read_icon(os.path.join(ICON_DIR, 'line_select_disabled.svg'), 'svg+xml')).tag(sync=True) # noqa
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@
></v-progress-circular>
</div>

<div v-if="line_items.length > 0">
<div v-if="line_menu_items.length > 0">
<j-plugin-section-header>Redshift from Centroid</j-plugin-section-header>
<v-row>
<j-docs-link>Assign the centroid reported above to the observed wavelength of a given line and set the resulting redshift. Lines must be loaded and plotted through the Line Lists plugin first.</j-docs-link>
Expand Down

0 comments on commit f667af7

Please sign in to comment.