Changes, style, test fixes

Author: snbianco

CHANGES.rst

@@ -23,19 +23,81 @@ simbad
 Service fixes and enhancements
 ------------------------------
 
-gaia
+linelists.cdms
+^^^^^^^^^^^^^^
+
+- Add whole catalog retrieval, improve error messaging for unparseable lines,
+  improve metadata catalog, and improve lookuptable behavior [#3173,#2901]
+
+heasarc
+^^^^^^^
+
+- Fix Heasarc.download_data for Sciserver. [#3183]
+
+jplspec
+^^^^^^^
+
+- minor improvement to lookuptable behavior [#3173,#2901]
+
+mast
 ^^^^
 
-- Update DR4 retrieval_type names and include the new one EPOCH_ASTROMETRY_BRIGHT [#3207]
+- Retrieve data products from the Missions-MAST API with ``mast.MastMissions.get_product_list``. Retrieve unique data
+  products only with ``mast.MastMissions.get_unique_product_list``. [#3155]
 
-ipac.irsa
+- Filter data products retrieved from the Missions-MAST API with ``mast.MastMissions.filter_products``. [#3155]
+
+- Download data products from the Missions-MAST API with ``mast.MastMissions.download_products``. 
+  Download a single data product using ``mast.MastMissions.download_file``. [#3155]
+
+- Get the keyword corresponding to the dataset ID for a specific mission with ``mast.MastMissions.get_dataset_kwd``. [#3155]
+
+- Handle coordinates that are not in the ICRS frame in query functions. [#3164]
+
+mocserver
 ^^^^^^^^^
 
-- Adding the "servicetype" kwarg to ``list_collections`` to be able to list SIA
-  and SSA collections separately. [#3200]
+- Switch to https instead of http for the default url (allows pyodide to use the
+  module) [#3139]
+
+- Add ``TimeMOC`` and ``STMOC`` as possible entries in ``MOCServer.query_region`` to
+  allow temporal and space-time searches [#3139]
+
+- ``return_moc`` now allows to ask for a Time-MOC or a Space-Time MOC rather than only
+  Space-MOCs [#3139]
+
+- Fix query by MOC that would write a file ``moc.fits`` where the method was executed in
+  overwriting mode (potentially deleting data if there was a conflicting file) [#3139]
+
+- [:warning: BREAKING] Returned tables now have a default list of fields instead of the
+  > 130 columns returned previously. The full list of fields can be displayed with the
+  new method ``MOCServer.list_fields`` [#3139]
+
+- Add ``casesensitive`` parameter in the queries (previously, this was hardcoded
+  to ``True``) [#3139]
+
+- Add ``coordinate_system`` parameter to the queries to allow to filter on the different
+  bodies or frames. The list of available space systems can be printed with the new
+  method ``MOCServer.list_coordinates_systems`` [#3139]
+
+- Add ``query_hips`` method, which is convenient to filter only Hierarchical progressive
+  surveys [#3139]
+
+- New parameter ``criteria`` in ``query_region`` and ``query_hips`` that has the same
+  use than ``meta_data`` in the deprecated method ``find_datasets`` [#3139]
+
+simbad
+^^^^^^
+
+- Fixed adding a list of fluxes with the deprecated notation
+  ``Simbad.add_votable_fields("flux(U)", "flux(J)")`` [#3186]
+
+- Support more of the 0.4.7 votable fields. Raise more significant error messages
+  for the discontinued ones. [#3186]
+
+- Fix the deprecated votable fields ``otype(V)`` and ``otype(S)`` [#3186]
 
-- Adding support for asynchronous queries using the new ``async_job``
-  keyword. [#3201]
+- Fixed non existing flux filters as votable fields would fail silently [#3186]
 
 ipac.nexsci.nasa_exoplanet_archive
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

astroquery/mast/missions.py

@@ -19,7 +19,7 @@
 from requests import HTTPError, RequestException
 
 from astroquery import log
-from astroquery.utils import commons, async_to_sync
+from astroquery.utils import async_to_sync
 from astroquery.utils.class_or_instance import class_or_instance
 from astroquery.exceptions import InvalidQueryError, MaxResultsWarning, InputWarning, NoResultsWarning
 

astroquery/mast/tests/test_mast_remote.py

@@ -1292,7 +1292,7 @@ def test_tesscut_get_sectors_mt(self):
         assert sector_table['ccd'][0] == 1
 
         error_noname = ("Please specify the object name or ID (as understood by the "
-                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons.cgi>`__) "
+                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons/app.html>`__) "
                         "of a moving target such as an asteroid or comet.")
         error_nameresolve = f"Could not resolve {moving_target_name} to a sky position."
         error_mt_coord = "Only one of moving_target and coordinates may be specified."
@@ -1375,7 +1375,7 @@ def test_tesscut_download_cutouts_mt(self, tmpdir):
             assert os.path.isfile(row['Local Path'])
 
         error_noname = ("Please specify the object name or ID (as understood by the "
-                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons.cgi>`__) of "
+                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons/app.html>`__) of "
                         "a moving target such as an asteroid or comet.")
         error_nameresolve = f"Could not resolve {moving_target_name} to a sky position."
         error_mt_coord = "Only one of moving_target and coordinates may be specified."
@@ -1456,7 +1456,7 @@ def test_tesscut_get_cutouts_mt(self):
         assert isinstance(cutout_hdus_list[0], fits.HDUList)
 
         error_noname = ("Please specify the object name or ID (as understood by the "
-                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons.cgi>`__) of "
+                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons/app.html>`__) of "
                         "a moving target such as an asteroid or comet.")
         error_nameresolve = f"Could not resolve {moving_target_name} to a sky position."
         error_mt_coord = "Only one of moving_target and coordinates may be specified."