update changelog, remove deprecated code for v3.2

Author: bhazelton

CHANGELOG.md

@@ -3,6 +3,8 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+## [3.2.0] - 2025-04-25
+
 ### Added
 - A new page to the docs labeled "Conventions" has been added.
 - The methods `Telescope.__add__` and `Telescope.__iadd__` have been added to allow
@@ -97,6 +99,24 @@ selection (if input to `polarizations` keyword was unordered).
 - The `x_orientation` attribute on `Telescope` and `UVBeam`.
 - The options `"e"` and `"n"` for elements of `feed_array` in `Telescope` and `UVBeam`.
 
+### Removed
+- The `future_array_shapes` attribute on `UVBase` objects.
+- The `use_future_array_shapes` keyword in various class methods.
+- The `use_future_array_shapes` method on `UVBase` objects.
+- Support for accessing the telescope-related metadata through their old attribute
+names on `UVData`, `UVCal` and `UVFlag` rather than via their attributes on the
+attached `Telescope` object (e.g. `UVData.telescope_name` -> `UVData.telescope.name`
+and `UVData.antenna_positions` -> `UVData.telescope.antenna_positions`).
+- Support for passing telescope-related metadata as separate parameters to `UVData.new` and
+`UVCal.new` rather than `Telescope` objects.
+- The `UVData.get_ENU_antpos` method in favor of `UVData.telescope.get_enu_antpos`.
+- The `Telescope.telescope_location` and `Telescope.telescope_name` attributes
+in favor of `Telescope.location` and `Telescope.name`.
+- The `get_telescope` function in favor of the `known_telescope_location` function
+and the `Telescope.from_known_telescopes` classmethod.
+- The KNOWN_TELESCOPE dict in favor of the `known_telescope_location` function
+and the `Telescope.from_known_telescopes` classmethod.
+
 ## [3.1.3] - 2025-01-13
 
 ### Added

docs/make_telescope.py

@@ -11,7 +11,7 @@
 from astropy.time import Time
 
 from pyuvdata import Telescope
-from pyuvdata.telescopes import _KNOWN_TELESCOPES
+from pyuvdata.telescopes import KNOWN_TELESCOPES
 
 
 def write_telescope_rst(write_file=None):
@@ -86,8 +86,8 @@ def write_telescope_rst(write_file=None):
         "We also provide a convenience function to get known telescope locations.\n\n"
     )
 
-    known_tel_use = copy.deepcopy(_KNOWN_TELESCOPES)
-    for tel, tel_dict in _KNOWN_TELESCOPES.items():
+    known_tel_use = copy.deepcopy(KNOWN_TELESCOPES)
+    for tel, tel_dict in KNOWN_TELESCOPES.items():
         if "location" in tel_dict:
             known_tel_use[tel]["location"] = (
                 tel_dict["location"].to_geodetic().__repr__()

src/pyuvdata/__init__.py

@@ -51,7 +51,6 @@ def branch_scheme(version):  # pragma: nocover
 from .analytic_beam import AiryBeam, GaussianBeam, ShortDipoleBeam, UniformBeam  # noqa
 from .beam_interface import BeamInterface  # noqa
 from .telescopes import Telescope  # noqa
-from .telescopes import get_telescope  # noqa  # NB: get_telescopes is deprecated
 from .uvbeam import UVBeam  # noqa
 from .uvcal import UVCal  # noqa
 from .uvdata import FastUVH5Meta  # noqa
@@ -70,7 +69,6 @@ def branch_scheme(version):  # pragma: nocover
     "ShortDipoleBeam",
     "UniformBeam",
     "Telescope",
-    "get_telescope",
 ]
 
 

src/pyuvdata/telescopes.py

@@ -7,7 +7,6 @@
 
 import os
 import warnings
-from collections.abc import Mapping
 from pathlib import Path
 from typing import Literal
 
@@ -22,7 +21,7 @@
 from .utils.tools import _strict_raise, slicify
 from .uvbase import UVBase
 
-__all__ = ["Telescope", "known_telescopes", "known_telescope_location", "get_telescope"]
+__all__ = ["Telescope", "known_telescopes", "known_telescope_location"]
 
 # We use astropy sites for telescope locations. The dict below is for
 # telescopes not in astropy sites, or to include extra information for a telescope.
@@ -32,7 +31,7 @@
 # Antenna positions can be specified via a csv file with the following columns:
 # "name" -- antenna name, "number" -- antenna number, "x", "y", "z" -- ECEF coordinates
 # relative to the telescope location.
-_KNOWN_TELESCOPES = {
+KNOWN_TELESCOPES = {
     "PAPER": {
         "location": EarthLocation.from_geodetic(
             lat=Angle("-30d43m17.5s"), lon=Angle("21d25m41.9s"), height=1073.0 * units.m
@@ -117,7 +116,7 @@
 
 # Define a (private) dictionary that tracks whether the user wants warnings
 # to be raised on updating known telescopes from params.
-_WARN_STATUS = {k.lower(): True for k in _KNOWN_TELESCOPES}
+_WARN_STATUS = {k.lower(): True for k in KNOWN_TELESCOPES}
 
 
 def ignore_telescope_param_update_warnings_for(tel: str):
@@ -147,45 +146,6 @@ def unignore_telescope_param_update_warnings_for(tel: str):
     _WARN_STATUS[tel] = True
 
 
-# Deprecation to handle accessing old keys of KNOWN_TELESCOPES
-class TelMapping(Mapping):
-    def __init__(self, mapping=()):
-        self._mapping = dict(mapping)
-
-    def __getitem__(self, key):
-        warnings.warn(
-            "Directly accessing the KNOWN_TELESCOPES dict is deprecated. If you "
-            "need a telescope location, use the known_telescope_location function. "
-            "For a full Telescope object use the classmethod "
-            "Telescope.from_known_telescopes. This will become an error in version 3.2",
-            DeprecationWarning,
-        )
-        if key in ["latitude", "longitude", "altitude", "center_xyz"]:
-            if key == "latitude":
-                return self._mapping["location"].lat.rad
-            if key == "longitude":
-                return self._mapping["location"].lon.rad
-            if key == "altitude":
-                return self._mapping["location"].height.to_value("m")
-            if key == "center_xyz":
-                return units.Quantity(self._mapping["location"].geocentric).to_value(
-                    "m"
-                )
-
-        return self._mapping[key]
-
-    def __len__(self):
-        return len(self._mapping)
-
-    def __iter__(self):
-        return iter(self._mapping)
-
-
-KNOWN_TELESCOPES = TelMapping(
-    (name, TelMapping(tel_dict)) for name, tel_dict in _KNOWN_TELESCOPES.items()
-)
-
-
 def known_telescopes():
     """
     Get list of known telescopes.
@@ -196,34 +156,10 @@ def known_telescopes():
         List of known telescope names.
     """
     astropy_sites = [site for site in EarthLocation.get_site_names() if site != ""]
-    known_telescopes = list(set(astropy_sites + list(_KNOWN_TELESCOPES.keys())))
+    known_telescopes = list(set(astropy_sites + list(KNOWN_TELESCOPES.keys())))
     return known_telescopes
 
 
-def get_telescope(telescope_name):
-    """
-    Get Telescope object for a telescope in telescope_dict. Deprecated.
-
-    Parameters
-    ----------
-    telescope_name : str
-        Name of a telescope
-
-    Returns
-    -------
-    Telescope object
-        The Telescope object associated with telescope_name.
-    """
-    warnings.warn(
-        "This method is deprecated and will be removed in version 3.2. If you "
-        "just need a telescope location, use the known_telescope_location function. "
-        "For a full Telescope object use the classmethod "
-        "Telescope.from_known_telescopes.",
-        DeprecationWarning,
-    )
-    return Telescope.from_known_telescopes(telescope_name, run_check=False)
-
-
 def known_telescope_location(name: str, return_citation: bool = False, **kwargs):
     """
     Get the location for a known telescope.
@@ -256,9 +192,9 @@ def known_telescope_location(name: str, return_citation: bool = False, **kwargs)
     else:
         telescope_dict = {}
         tel_name = name.lower()
-        for key in _KNOWN_TELESCOPES:
+        for key in KNOWN_TELESCOPES:
             if key.lower() == tel_name:
-                telescope_dict = _KNOWN_TELESCOPES[key]
+                telescope_dict = KNOWN_TELESCOPES[key]
         for key in kwargs:
             if (key == "citation") or (key == "location"):
                 telescope_dict[key] = kwargs[key]
@@ -499,22 +435,7 @@ def __init__(self):
 
     def __getattr__(self, __name):
         """Handle old names attributes."""
-        if __name == "telescope_location":
-            warnings.warn(
-                "The Telescope.telescope_location attribute is deprecated, use "
-                "Telescope.location instead (which contains an astropy "
-                "EarthLocation object). This will become an error in version 3.2.",
-                DeprecationWarning,
-            )
-            return self._location.xyz()
-        elif __name == "telescope_name":
-            warnings.warn(
-                "The Telescope.telescope_name attribute is deprecated, use "
-                "Telescope.name instead. This will become an error in version 3.2.",
-                DeprecationWarning,
-            )
-            return self.name
-        elif __name == "x_orientation":
+        if __name == "x_orientation":
             warnings.warn(
                 "The Telescope.x_orientation attribute is deprecated, and has "
                 "been superseded by Telescope.feed_angle and Telescope.feed_array. "
@@ -529,22 +450,7 @@ def __getattr__(self, __name):
 
     def __setattr__(self, __name, __value):
         """Handle old names for telescope metadata."""
-        if __name == "telescope_location":
-            warnings.warn(
-                "The Telescope.telescope_location attribute is deprecated, use "
-                "Telescope.location instead (which should be set to an astropy "
-                "EarthLocation object). This will become an error in version 3.2.",
-                DeprecationWarning,
-            )
-            self._location.set_xyz(__value)
-        elif __name == "telescope_name":
-            warnings.warn(
-                "The Telescope.telescope_name attribute is deprecated, use "
-                "Telescope.name instead. This will become an error in version 3.2.",
-                DeprecationWarning,
-            )
-            self.name = __value
-        elif __name == "x_orientation":
+        if __name == "x_orientation":
             warnings.warn(
                 "The Telescope.x_orientation attribute is deprecated, and has "
                 "been superseded by Telescope.feed_angle and Telescope.feed_array. "
@@ -837,9 +743,9 @@ def update_params_from_known_telescopes(
             )
         telescope_dict = {}
         tel_name = self.name.lower()
-        for key in _KNOWN_TELESCOPES:
+        for key in KNOWN_TELESCOPES:
             if key.lower() == tel_name:
-                telescope_dict.update(_KNOWN_TELESCOPES[key])
+                telescope_dict.update(KNOWN_TELESCOPES[key])
 
         astropy_sites_list = []
         known_telescope_list = []

src/pyuvdata/utils/__init__.py

@@ -5,8 +5,6 @@
 
 from __future__ import annotations
 
-import warnings
-
 import numpy as np
 
 # standard angle tolerance: 1 mas in radians.
@@ -44,81 +42,3 @@
 from .pol import *  # noqa
 from .times import get_lst_for_time  # noqa
 from .uvcalibrate import uvcalibrate  # noqa
-
-# deprecated imports
-
-
-def _check_histories(history1, history2):
-    """Check if two histories are the same.
-
-    Deprecated. Use pyuvdata.utils.history._check_histories
-    """
-    from .history import _check_histories
-
-    warnings.warn(
-        "The _check_histories function has moved, please import it from "
-        "pyuvdata.utils.history. This warnings will become an error in version 3.2",
-        DeprecationWarning,
-    )
-
-    return _check_histories(history1, history2)
-
-
-def _fits_gethduaxis(hdu, axis):
-    """
-    Make axis arrays for fits files.
-
-    Deprecated. Use pyuvdata.utils.io.fits._gethduaxis.
-
-    Parameters
-    ----------
-    hdu : astropy.io.fits HDU object
-        The HDU to make an axis array for.
-    axis : int
-        The axis number of interest (1-based).
-
-    Returns
-    -------
-    ndarray of float
-        Array of values for the specified axis.
-
-    """
-    from .io.fits import _gethduaxis
-
-    warnings.warn(
-        "The _fits_gethduaxis function has moved, please import it as "
-        "pyuvdata.utils.io.fits._gethduaxis. This warnings will become an "
-        "error in version 3.2",
-        DeprecationWarning,
-    )
-
-    return _gethduaxis(hdu, axis)
-
-
-def _fits_indexhdus(hdulist):
-    """
-    Get a dict of table names and HDU numbers from a FITS HDU list.
-
-    Deprecated. Use pyuvdata.utils.io.fits._indexhdus.
-
-    Parameters
-    ----------
-    hdulist : list of astropy.io.fits HDU objects
-        List of HDUs to get names for
-
-    Returns
-    -------
-    dict
-        dictionary with table names as keys and HDU number as values.
-
-    """
-    from .io.fits import _indexhdus
-
-    warnings.warn(
-        "The _fits_indexhdus function has moved, please import it as "
-        "pyuvdata.utils.io.fits._indexhdus. This warnings will become an "
-        "error in version 3.2",
-        DeprecationWarning,
-    )
-
-    return _indexhdus(hdulist)

src/pyuvdata/utils/pol.py

@@ -24,7 +24,6 @@
     "jstr2num",
     "jnum2str",
     "conj_pol",
-    "_x_orientation_rep_dict",
     "x_orientation_pol_map",
     "parse_polstr",
     "parse_jpolstr",
@@ -86,23 +85,6 @@
 }
 
 
-def _x_orientation_rep_dict(x_orientation):
-    """
-    Create replacement dict based on x_orientation.
-
-    Deprecated. Use x_orientation_pol_map instead.
-
-    """
-    warnings.warn(
-        "This function (_x_orientation_rep_dict) is deprecated, use "
-        "pyuvdata.utils.pol.x_orientation_pol_map instead."
-        " This will become an error in version 3.2",
-        DeprecationWarning,
-    )
-
-    return x_orientation_pol_map(x_orientation)
-
-
 def x_orientation_pol_map(x_orientation: str) -> dict:
     """
     Return map from "x" and "y" pols to "e" and "n" based on x_orientation.