Skip to content

Commit

Permalink
Fix PEP 8 warning
Browse files Browse the repository at this point in the history
and remove outdated comment
  • Loading branch information
pllim committed Aug 10, 2023
1 parent 938253d commit d62084f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 7 deletions.
7 changes: 4 additions & 3 deletions jdaviz/configs/default/plugins/subset_plugin/subset_plugin.py
Original file line number Diff line number Diff line change
Expand Up @@ -159,9 +159,10 @@ def _unpack_get_subsets_for_ui(self):
if isinstance(subset_state.roi, CircularROI):
x, y = subset_state.roi.center()
r = subset_state.roi.radius
subset_definition += [{"name": "X Center", "att": "xc", "value": x, "orig": x},
{"name": "Y Center", "att": "yc", "value": y, "orig": y},
{"name": "Radius", "att": "radius", "value": r, "orig": r}]
subset_definition += [
{"name": "X Center", "att": "xc", "value": x, "orig": x},
{"name": "Y Center", "att": "yc", "value": y, "orig": y},
{"name": "Radius", "att": "radius", "value": r, "orig": r}]

elif isinstance(subset_state.roi, RectangularROI):
for att in ("Xmin", "Xmax", "Ymin", "Ymax"):
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,6 @@

ASTROPY_LT_5_2 = Version(astropy.__version__) < Version('5.2')

# TODO: This plugin needs to show params wrt ref data because that is how glue
# defined it, but then it needs to do internal calculation using region
# that took account of the dither.


@tray_registry('imviz-aper-phot-simple', label="Imviz Simple Aperture Photometry")
class SimpleAperturePhotometry(PluginTemplateMixin, DatasetSelectMixin, TableMixin, PlotMixin):
Expand Down

0 comments on commit d62084f

Please sign in to comment.