Fix numpy deprecation warning by switching to use Table.read for reading response

Author: bsipocz

astroquery/sdss/core.py

@@ -896,21 +896,15 @@ def _parse_result(self, response, verbose=False):
         table : `~astropy.table.Table`
 
         """
+        if 'error_message' in response.text:
+            raise RemoteServiceError(response.text)
 
-        if 'error_message' in io.BytesIO(response.content):
-            raise RemoteServiceError(response.content)
-        skip_header = 0
-        if response.content.startswith(b'#Table'):
-            skip_header = 1
-        arr = np.atleast_1d(np.genfromtxt(io.BytesIO(response.content),
-                                          names=True, dtype=None,
-                                          delimiter=',', skip_header=skip_header,
-                                          comments='#'))
+        arr = Table.read(response.text, format='ascii.csv', comment="#")
 
         if len(arr) == 0:
             return None
         else:
-            return Table(arr)
+            return arr
 
     def _args_to_payload(self, coordinates=None,
                          fields=None, spectro=False, region=False,

astroquery/sdss/tests/test_sdss.py

@@ -144,7 +144,7 @@ def url_tester_crossid(data_release):
 
 def compare_xid_data(xid, data):
     for col in xid.colnames:
-        if xid[col].dtype.type is np.string_:
+        if xid[col].dtype.type is np.str_:
             assert xid[col] == data[col]
         else:
             assert_allclose(xid[col], data[col])

setup.cfg

@@ -39,7 +39,6 @@ filterwarnings =
 # This is a temporary measure, all of these should be fixed:
     ignore:distutils Version classes are deprecated:DeprecationWarning
     ignore::pytest.PytestUnraisableExceptionWarning
-    ignore::numpy.VisibleDeprecationWarning
     ignore:unclosed <ssl.SSLSocket:ResourceWarning
     ignore::UserWarning
     ignore::astroquery.exceptions.InputWarning