ENH: address copilot comments

Author: MateusStano

rocketpy/environment/environment.py

@@ -178,14 +178,14 @@ class Environment:
         ``Ensemble``.
     Environment.lat_array : array
         Defined if ``netCDF`` or ``OPeNDAP`` file is used, for Forecasts,
-        Reanalysis and Ensembles. 2x2 matrix for each pressure level of
-        latitudes corresponding to the vertices of the grid cell which
-        surrounds the launch site.
+        Reanalysis and Ensembles. Two-element list ``[x1, x2]`` containing
+        the latitude coordinates of the grid-cell vertices that bracket the
+        launch site and are used in bilinear interpolation.
     Environment.lon_array : array
         Defined if ``netCDF`` or ``OPeNDAP`` file is used, for Forecasts,
-        Reanalysis and Ensembles. 2x2 matrix for each pressure level of
-        longitudes corresponding to the vertices of the grid cell which
-        surrounds the launch site.
+        Reanalysis and Ensembles. Two-element list ``[y1, y2]`` containing
+        the longitude coordinates of the grid-cell vertices that bracket the
+        launch site and are used in bilinear interpolation.
     Environment.lon_index : int
         Defined if ``netCDF`` or ``OPeNDAP`` file is used, for Forecasts,
         Reanalysis and Ensembles. Index to a grid longitude which
@@ -223,7 +223,8 @@ class Environment:
         surrounds the launch site.
     Environment.time_array : array
         Defined if ``netCDF`` or ``OPeNDAP`` file is used, for Forecasts,
-        Reanalysis and Ensembles. Array of dates available in the file.
+        Reanalysis and Ensembles. Two-element list with the first and last
+        values from the dataset time variable in the dataset native units.
     Environment.height : array
         Defined if ``netCDF`` or ``OPeNDAP`` file is used, for Forecasts,
         Reanalysis and Ensembles. List of geometric height corresponding to

rocketpy/environment/fetchers.py

@@ -93,8 +93,8 @@ def fetch_atmospheric_data_from_windy(lat, lon, model):
 
 
 def fetch_gfs_file_return_dataset(max_attempts=10, base_delay=2):
-    """Fetches the latest GFS (Global Forecast System) dataset from the NOAA's
-    GrADS data server using the OpenDAP protocol.
+    """Fetches the latest GFS (Global Forecast System) dataset from the UCAR
+    THREDDS data server using the OPeNDAP protocol.
 
     Parameters
     ----------
@@ -128,8 +128,8 @@ def fetch_gfs_file_return_dataset(max_attempts=10, base_delay=2):
 
 
 def fetch_nam_file_return_dataset(max_attempts=10, base_delay=2):
-    """Fetches the latest NAM (North American Mesoscale) dataset from the NOAA's
-    GrADS data server using the OpenDAP protocol.
+    """Fetches the latest NAM (North American Mesoscale) dataset from the UCAR
+    THREDDS data server using the OPeNDAP protocol.
 
     Parameters
     ----------
@@ -161,8 +161,8 @@ def fetch_nam_file_return_dataset(max_attempts=10, base_delay=2):
 
 
 def fetch_rap_file_return_dataset(max_attempts=10, base_delay=2):
-    """Fetches the latest RAP (Rapid Refresh) dataset from the NOAA's GrADS data
-    server using the OpenDAP protocol.
+    """Fetches the latest RAP (Rapid Refresh) dataset from the UCAR THREDDS
+    data server using the OPeNDAP protocol.
 
     Parameters
     ----------
@@ -193,78 +193,6 @@ def fetch_rap_file_return_dataset(max_attempts=10, base_delay=2):
     raise RuntimeError("Unable to load latest weather data for RAP through " + file_url)
 
 
-def fetch_hrrr_file_return_dataset(max_attempts=10, base_delay=2):
-    """Fetches the latest HRRR (High-Resolution Rapid Refresh) dataset from
-    the NOAA's GrADS data server using the OpenDAP protocol.
-
-    Parameters
-    ----------
-    max_attempts : int, optional
-        The maximum number of attempts to fetch the dataset. Default is 10.
-    base_delay : int, optional
-        The base delay in seconds between attempts. Default is 2.
-
-    Returns
-    -------
-    netCDF4.Dataset
-        The HRRR dataset.
-
-    Raises
-    ------
-    RuntimeError
-        If unable to load the latest weather data for HRRR.
-    """
-    file_url = "https://thredds.ucar.edu/thredds/dodsC/grib/NCEP/HRRR/CONUS_2p5km/Best"
-    attempt_count = 0
-    while attempt_count < max_attempts:
-        try:
-            return netCDF4.Dataset(file_url)
-        except OSError:
-            attempt_count += 1
-            time.sleep(base_delay**attempt_count)
-
-    raise RuntimeError(
-        "Unable to load latest weather data for HRRR through " + file_url
-    )
-
-
-def fetch_aigfs_file_return_dataset(max_attempts=10, base_delay=2):
-    """Fetches the latest AIGFS (Artificial Intelligence GFS) dataset from
-    the NOAA's GrADS data server using the OpenDAP protocol.
-
-    Parameters
-    ----------
-    max_attempts : int, optional
-        The maximum number of attempts to fetch the dataset. Default is 10.
-    base_delay : int, optional
-        The base delay in seconds between attempts. Default is 2.
-
-    Returns
-    -------
-    netCDF4.Dataset
-        The AIGFS dataset.
-
-    Raises
-    ------
-    RuntimeError
-        If unable to load the latest weather data for AIGFS.
-    """
-    file_url = (
-        "https://thredds.ucar.edu/thredds/dodsC/grib/NCEP/AIGFS/Global_0p25deg/Best"
-    )
-    attempt_count = 0
-    while attempt_count < max_attempts:
-        try:
-            return netCDF4.Dataset(file_url)
-        except OSError:
-            attempt_count += 1
-            time.sleep(base_delay**attempt_count)
-
-    raise RuntimeError(
-        "Unable to load latest weather data for AIGFS through " + file_url
-    )
-
-
 def fetch_hiresw_file_return_dataset(max_attempts=10, base_delay=2):
     """Fetches the latest HiResW (High-Resolution Window) dataset from the NOAA's
     GrADS data server using the OpenDAP protocol.

rocketpy/environment/weather_model_mapping.py

@@ -7,21 +7,21 @@ class WeatherModelMapping:
     Each dictionary in this class maps those canonical keys to the actual
     variable names in a specific data provider format.
 
-        Mapping families
-        ----------------
-        - Base names (for example ``GFS``, ``NAM``, ``RAP``) represent the current
-            default mappings used by the latest-model shortcuts and THREDDS-style
-            datasets.
-        - ``*_LEGACY`` names represent older NOMADS-style variable naming
-            conventions (for example ``lev``, ``tmpprs``, ``ugrdprs`` and
-            ``vgrdprs``) and are intended for archived or previously downloaded files.
+    Mapping families
+    ----------------
+    - Base names (for example ``GFS``, ``NAM``, ``RAP``) represent the current
+      default mappings used by the latest-model shortcuts and THREDDS-style
+      datasets.
+    - ``*_LEGACY`` names represent older NOMADS-style variable naming
+      conventions (for example ``lev``, ``tmpprs``, ``ugrdprs`` and
+      ``vgrdprs``) and are intended for archived or previously downloaded files.
 
-        Notes
-        -----
-        - Mappings can also include optional keys such as ``projection`` for
-            projected grids and ``ensemble`` for member dimensions.
-        - The :meth:`get` method is case-insensitive, so ``"gfs_legacy"`` and
-            ``"GFS_LEGACY"`` are equivalent.
+    Notes
+    -----
+    - Mappings can also include optional keys such as ``projection`` for
+      projected grids and ``ensemble`` for member dimensions.
+    - The :meth:`get` method is case-insensitive, so ``"gfs_legacy"`` and
+      ``"GFS_LEGACY"`` are equivalent.
     """
 
     GFS = {