Skip to content

Commit b8955d2

Browse files
committed
fix: ucd keyword had a typo
1 parent 1c03261 commit b8955d2

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

CHANGES.rst

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,11 @@ skyview
149149
- Overlay arguments ``lut``, ``grid``, and ``gridlabel`` are removed, as they
150150
only apply to output types not returned by Astroquery [#2979]
151151

152+
vizier
153+
^^^^^^
154+
155+
- Fixed search by UCD -- they were ignored. [#3147]
156+
152157
vsa
153158
^^^
154159

astroquery/vizier/core.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ def __init__(self, *, columns=["*"], column_filters={}, catalog=None,
7878
self.column_filters = column_filters
7979
self.catalog = catalog
8080
self._keywords = None
81-
self.ucd = ucd
81+
self._ucd = ucd
8282
if keywords:
8383
self.keywords = keywords
8484
self.TIMEOUT = timeout
@@ -233,7 +233,7 @@ def find_catalogs(self, keywords, *, include_obsolete=False, verbose=False,
233233
data_payload["-obsolete"] = None
234234

235235
if self.ucd != "":
236-
data_payload["ucd"] = self.ucd
236+
data_payload["-ucd"] = self.ucd
237237

238238
params = "&".join([k if v is None else f"{k}={v}" for k, v in data_payload.items()])
239239

astroquery/vizier/tests/test_vizier_remote.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,6 @@ def test_findcatalog_maxcatalog(self):
131131
assert len(cats) >= 39 # as of 2024
132132

133133
def test_findcatalog_ucd(self):
134-
# this fails for VizieR 7.33.3, should work in next releases
135134
vizier = Vizier()
136135
ucdresult = vizier(ucd='phys.albedo').find_catalogs('mars', max_catalogs=5000)
137136
result = vizier.find_catalogs('mars', max_catalogs=5000)

0 commit comments

Comments
 (0)