MAINT: skybox to use votable responses to workaround parsing non-ascii objectnames from responses

Author: bsipocz

astroquery/imcce/__init__.py

@@ -31,45 +31,30 @@ class Conf(_config.ConfigNamespace):
     # SkyBoT configuration
 
     # dictionary for field name and unit conversions using 'output=all`
-    field_names = {'Num': 'Number',
-                   'Name': 'Name',
-                   'RA(h)': 'RA',
-                   'DE(deg)': 'DEC',
-                   'Class': 'Type',
-                   'Mv': 'V',
-                   'Err(arcsec)': 'posunc',
-                   'd(arcsec)': 'centerdist',
-                   'dRA(arcsec/h)': 'RA_rate',
-                   'dDEC(arcsec/h)': 'DEC_rate',
-                   'Dg(ua)': 'geodist',
-                   'Dh(ua)': 'heliodist',
-                   'Phase(deg)': 'alpha',
-                   'SunElong(deg)': 'elong',
-                   'x(au)': 'x',
-                   'y(au)': 'y',
-                   'z(au)': 'z',
-                   'vx(au/d)': 'vx',
-                   'vy(au/d)': 'vy',
-                   'vz(au/d)': 'vz',
-                   'Ref. Epoch(JD)': 'epoch'}
-    field_units = {'RA': u.deg,  # after conversion to deg
-                   'DEC': u.deg,  # after conversion to deg
-                   'V': u.mag,
-                   'posunc': u.arcsec,
-                   'centerdist': u.arcsec,
-                   'RA_rate': u.arcsec/u.hour,
-                   'DEC_rate': u.arcsec/u.hour,
-                   'geodist': u.au,
-                   'heliodist': u.au,
-                   'alpha': u.degree,
-                   'elong': u.degree,
-                   'x': u.au,
-                   'y': u.au,
-                   'z': u.au,
-                   'vx': u.au/u.day,
-                   'vy': u.au/u.day,
-                   'vz': u.au/u.day,
-                   'epoch': u.day}
+    field_names = {'num': 'Number',
+                   'name': 'Name',
+                   'ra': 'RA',
+                   'de': 'DEC',
+                   'class': 'Type',
+                   'magV': 'V',
+                   'errpos': 'posunc',
+                   'angdist': 'centerdist',
+                   'dracosdec': 'RA_rate',
+                   'ddec': 'DEC_rate',
+                   'dgeo': 'geodist',
+                   'dhelio': 'heliodist',
+                   'phase': 'alpha',
+                   'solelong': 'elong',
+                   'px': 'x',
+                   'py': 'y',
+                   'pz': 'z',
+                   'vx': 'vx',
+                   'vy': 'vy',
+                   'vz': 'vz',
+                   'jdref': 'epoch',
+                   '_raj2000': '_raj2000',
+                   '_decj2000': '_decj2000',
+                   'externallink': 'externallink'}
 
 
 conf = Conf()

astroquery/imcce/core.py

@@ -599,7 +599,7 @@ def cone_search_async(self,
                            + str(int(find_comets)),
                            '-refsys': 'EQJ2000',
                            '-output': 'all',
-                           '-mime': 'text'}
+                           '-mime': 'votable'}
 
         # check for diagnostic flags
         if get_query_payload:
@@ -617,36 +617,26 @@ def cone_search_async(self,
 
     def _parse_result(self, response, *, verbose=False):
         """
-        internal wrapper to parse queries
+        internal wrapper to parse queries.
         """
 
         if self._get_raw_response:
             return response.text
 
-        # intercept error messages
-        response_txt = response.text.split('\n')[2:-1]
-        if len(response_txt) < 3 and len(response_txt[-1].split('|')) < 21:
-            raise RuntimeError(response_txt[-1])
-
-        names = response_txt[0].replace('# ', '').strip().split(' | ')
-        results = ascii.read(response_txt[1:], delimiter='|',
-                             names=names, fast_reader=False)
-        results = QTable(results)
+        results = QTable.read(BytesIO(response.content), format='votable')
 
         # convert coordinates to degrees
-        coo = SkyCoord(ra=results['RA(h)'], dec=results['DE(deg)'],
+        coo = SkyCoord(ra=results['ra'], dec=results['de'],
                        unit=(u.hourangle, u.deg), frame='icrs')
-        results['RA(h)'] = coo.ra.deg
-        results['DE(deg)'] = coo.dec.deg
+        results['ra'] = coo.ra.deg
+        results['ra'].unit = u.deg
+        results['de'] = coo.dec.deg
+        results['de'].unit = u.deg
 
         colnames = results.columns[:]
         for fieldname in colnames:
             # apply field name change
             results.rename_column(fieldname, conf.field_names[fieldname])
-            # apply unit, if available
-            if conf.field_names[fieldname] in conf.field_units:
-                results[conf.field_names[fieldname]].unit = conf.field_units[
-                    conf.field_names[fieldname]]
 
         # convert object numbers to int
         # unnumbered asteroids return as non numeric values ('-')