no no... the default was what it was supposed to be. But the test

works.  Splatalogue server no longer supports version selection

Author: keflavich

astroquery/splatalogue/core.py

@@ -123,7 +123,7 @@ def _parse_kwargs(self, min_frequency=None, max_frequency=None,
                       chem_re_flags=0, energy_min=None, energy_max=None,
                       energy_type=None, intensity_lower_limit=None,
                       intensity_type=None, transition=None, version=None,
-                      exclude='none', only_NRAO_recommended=None,
+                      exclude=None, only_NRAO_recommended=None,
                       line_lists=None, line_strengths=None, energy_levels=None,
                       export=None, export_limit=None, noHFS=None,
                       displayHFS=None, show_unres_qn=None,

astroquery/splatalogue/tests/test_splatalogue.py

@@ -107,23 +107,25 @@ def test_band_crashorno():
     assert exc.value.args[0] == "Invalid frequency band."
 
 
-# regression test : version selection should work
-# Unfortunately, it looks like version1 = version2 on the web page now, so this
-# may no longer be a valid test
-@remote_data
-def test_version_selection():
-    results = splatalogue.Splatalogue.query_lines(
-			    min_frequency= 703*u.GHz,
-			    max_frequency=706*u.GHz,
-			    chemical_name='Acetaldehyde',
-			    version='v1.0'
-			    )
-    assert len(results)==1
+#Upstream changed: there is no distinction between versions for this molecule
+# # regression test : version selection should work
+# # Unfortunately, it looks like version1 = version2 on the web page now, so this
+# # may no longer be a valid test
+# @remote_data
+# def test_version_selection():
+#     results = splatalogue.Splatalogue.query_lines(
+# 			    min_frequency= 703*u.GHz,
+# 			    max_frequency=706*u.GHz,
+# 			    chemical_name='Acetaldehyde',
+# 			    version='v1.0'
+# 			    )
+#     assert len(results)==1
 
 def test_exclude(patch_post):
     # regression test for issue 616
     d = splatalogue.Splatalogue.query_lines_async(114 * u.GHz, 116 * u.GHz,
                                                   chemical_name=' CO ',
+                                                  exclude=None,
                                                   get_query_payload=True)
 
     exclusions = {'no_atmospheric': 'no_atmospheric',
@@ -155,7 +157,7 @@ def test_exclude_remote():
         energy_type='eu_k',
         line_lists=['CDMS'],
         export_limit=1000.,
-        exclude=None,
+        exclude='none',
         version='v2.0',
         show_upper_degeneracy=True,
         )