Skip to content

Commit ca9a4a5

Browse files
mwcraigCopilot
andcommitted
Apply suggestions from code review
Co-authored-by: Copilot <[email protected]>
1 parent c58e9c9 commit ca9a4a5

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

src/astro_image_display_api/dummy_viewer.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,7 @@ def remove_catalog(self, catalog_label: str | None = None) -> None:
365365
try:
366366
del self._catalogs[catalog_label]
367367
except KeyError:
368-
raise ValueError(f"Marker name {catalog_label} not found.")
368+
raise ValueError(f"Catalog label {catalog_label} not found.")
369369

370370
def get_catalog(self, x_colname: str = 'x', y_colname: str = 'y',
371371
skycoord_colname: str = 'coord',

src/astro_image_display_api/interface_definition.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ def load_catalog(self, table: Table, x_colname: str = 'x', y_colname: str = 'y',
151151
name will be generated.
152152
catalog_style: dict, optional
153153
A dictionary that specifies the style of the markers used to
154-
rerpresent the catalog. See `ImageViewerInterface.set_catalog_style`
154+
represent the catalog. See `ImageViewerInterface.set_catalog_style`
155155
for details.
156156
"""
157157
raise NotImplementedError

src/astro_image_display_api/widget_api_test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ def test_zoom(self):
137137
assert self.image.zoom_level == 6 # 3 x 2
138138

139139
def test_set_catalog_style_before_catalog_data_raises_error(self):
140-
# Make sure3 that adding a catalog style before adding any catalog
140+
# Make sure that adding a catalog style before adding any catalog
141141
# data raises an error.
142142
with pytest.raises(
143143
ValueError,
@@ -299,7 +299,7 @@ def test_load_catalog_multiple_same_label(self, catalog):
299299

300300
def test_load_catalog_with_skycoord_no_wcs(self, catalog, data):
301301
# Check that loading a catalog with skycoord but no x/y and
302-
# no WCS returns a catlog with None for x and y.
302+
# no WCS returns a catalog with None for x and y.
303303
self.image.load_array(data)
304304

305305
# Remove x/y columns from the catalog

0 commit comments

Comments
 (0)