Skip to content

Commit

Permalink
Changes, style, test fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
snbianco committed Jan 24, 2025
1 parent a35fe9a commit 50648fd
Show file tree
Hide file tree
Showing 3 changed files with 83 additions and 4 deletions.
79 changes: 79 additions & 0 deletions CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,85 @@ New Tools and Services
Service fixes and enhancements
------------------------------

<<<<<<< HEAD
=======
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
^^^^

- 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]

- 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
^^^^^^^^^

- 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]

- Fixed non existing flux filters as votable fields would fail silently [#3186]

>>>>>>> 1b2123ae (Changes, style, test fixes)
ipac.nexsci.nasa_exoplanet_archive
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Expand Down
2 changes: 1 addition & 1 deletion astroquery/mast/missions.py
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
6 changes: 3 additions & 3 deletions astroquery/mast/tests/test_mast_remote.py
Original file line number Diff line number Diff line change
Expand Up @@ -1262,7 +1262,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."
Expand Down Expand Up @@ -1345,7 +1345,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."
Expand Down Expand Up @@ -1426,7 +1426,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."
Expand Down

0 comments on commit 50648fd

Please sign in to comment.