Skip to content

Commit dc95628

Browse files
mwcraigCopilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot <[email protected]>
1 parent ee57bd9 commit dc95628

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
@@ -353,7 +353,7 @@ def remove_catalog(self, catalog_label: str | None = None) -> None:
353353
try:
354354
del self._catalogs[catalog_label]
355355
except KeyError:
356-
raise ValueError(f"Marker name {catalog_label} not found.")
356+
raise ValueError(f"Catalog label {catalog_label} not found.")
357357

358358
def get_catalog(self, x_colname: str = 'x', y_colname: str = 'y',
359359
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
@@ -124,7 +124,7 @@ def load_catalog(self, table: Table, x_colname: str = 'x', y_colname: str = 'y',
124124
name will be generated.
125125
catalog_style: dict, optional
126126
A dictionary that specifies the style of the markers used to
127-
rerpresent the catalog. See `ImageViewerInterface.set_catalog_style`
127+
represent the catalog. See `ImageViewerInterface.set_catalog_style`
128128
for details.
129129
"""
130130
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
@@ -135,7 +135,7 @@ def test_zoom(self):
135135
assert self.image.zoom_level == 6 # 3 x 2
136136

137137
def test_set_catalog_style_before_catalog_data_raises_error(self):
138-
# Make sure3 that adding a catalog style before adding any catalog
138+
# Make sure that adding a catalog style before adding any catalog
139139
# data raises an error.
140140
with pytest.raises(
141141
ValueError,
@@ -297,7 +297,7 @@ def test_load_catalog_multiple_same_label(self, catalog):
297297

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

303303
# Remove x/y columns from the catalog

0 commit comments

Comments
 (0)