From d62084ff8ef8031e02dc3f4b90b8413fa3ead1db Mon Sep 17 00:00:00 2001
From: "P. L. Lim" <2090236+pllim@users.noreply.github.com>
Date: Tue, 8 Aug 2023 18:12:48 -0400
Subject: [PATCH] Fix PEP 8 warning and remove outdated comment

---
 .../configs/default/plugins/subset_plugin/subset_plugin.py | 7 ++++---
 .../imviz/plugins/aper_phot_simple/aper_phot_simple.py     | 4 ----
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/jdaviz/configs/default/plugins/subset_plugin/subset_plugin.py b/jdaviz/configs/default/plugins/subset_plugin/subset_plugin.py
index 4ac9a9f43a..8e08b58b6d 100644
--- a/jdaviz/configs/default/plugins/subset_plugin/subset_plugin.py
+++ b/jdaviz/configs/default/plugins/subset_plugin/subset_plugin.py
@@ -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"):
diff --git a/jdaviz/configs/imviz/plugins/aper_phot_simple/aper_phot_simple.py b/jdaviz/configs/imviz/plugins/aper_phot_simple/aper_phot_simple.py
index 79872b3068..2e1e9b4521 100644
--- a/jdaviz/configs/imviz/plugins/aper_phot_simple/aper_phot_simple.py
+++ b/jdaviz/configs/imviz/plugins/aper_phot_simple/aper_phot_simple.py
@@ -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):