Skip to content

Commit 9870657

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

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
@@ -375,7 +375,7 @@ def remove_catalog(self, catalog_label: str | None = None) -> None:
375375
try:
376376
del self._catalogs[catalog_label]
377377
except KeyError:
378-
raise ValueError(f"Marker name {catalog_label} not found.")
378+
raise ValueError(f"Catalog label {catalog_label} not found.")
379379

380380
def get_catalog(self, x_colname: str = 'x', y_colname: str = 'y',
381381
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
@@ -102,7 +102,7 @@ def load_catalog(self, table: Table, x_colname: str = 'x', y_colname: str = 'y',
102102
name will be generated.
103103
catalog_style: dict, optional
104104
A dictionary that specifies the style of the markers used to
105-
rerpresent the catalog. See `ImageViewerInterface.set_catalog_style`
105+
represent the catalog. See `ImageViewerInterface.set_catalog_style`
106106
for details.
107107
"""
108108
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
@@ -136,7 +136,7 @@ def test_zoom(self):
136136
assert self.image.zoom_level == 6 # 3 x 2
137137

138138
def test_set_catalog_style_before_catalog_data_raises_error(self):
139-
# Make sure3 that adding a catalog style before adding any catalog
139+
# Make sure that adding a catalog style before adding any catalog
140140
# data raises an error.
141141
with pytest.raises(
142142
ValueError,
@@ -298,7 +298,7 @@ def test_load_catalog_multiple_same_label(self, catalog):
298298

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

304304
# Remove x/y columns from the catalog

0 commit comments

Comments
 (0)