MNT: Attempt to remove pinning and fixes for ruff errors. (#1666)

* MNT: Remove pinning from requirements.

* MNT: Remove pinning from environment file.

* STY: PEP8 fixes.

* STY: Fix for misordered imports.

* STY: Revert old check.

* CI: Update wheels.yml to use latest OS.

.github/workflows/wheels.yml

@@ -26,8 +26,8 @@ jobs:
     strategy:
       matrix:
         os: [
-          ubuntu-20.04,
-          windows-2019,
+          ubuntu-latest,
+          windows-latest,
           macos-14,
           macos-13,
         ]

continuous_integration/environment-ci.yml

@@ -24,7 +24,7 @@ dependencies:
   - cftime
   - setuptools
   - shapely
-  - ruff==0.4.2
+  - ruff
   - mda-xdrlib
   - xradar
   - pooch

examples/plotting/radar-cross-section.ipynb

@@ -7,6 +7,7 @@
    "metadata": {},
    "outputs": [],
    "source": [
+    "import cartopy.crs as ccrs\n",
     "import matplotlib.pyplot as plt\n",
     "\n",
     "import pyart\n",

pyart/graph/radarmapdisplay_basemap.py

@@ -288,7 +288,7 @@ def plot_ppi_map(
             data = np.ma.masked_outside(data, vmin, vmax)
 
         # create the basemap if not provided
-        if type(basemap) != Basemap:
+        if not isinstance(basemap, Basemap):
             using_corners = None not in [min_lon, min_lat, max_lon, max_lat]
             if using_corners:
                 basemap = Basemap(

requirements.txt

@@ -1,15 +1,15 @@
 numpy
 scipy
-netCDF4>=1.7.0
+netCDF4
 matplotlib
 pooch
 cftime
 fsspec
 s3fs
 open_radar_data
-xradar>=0.5.0
+xradar
 pandas
 mda-xdrlib
-xarray!=0.21.0
+xarray
 cartopy
 pint

tests/io/test_cfradial.py

@@ -486,7 +486,7 @@ def attribute_equal(class1, class2, key, allow_str_case_diff=True):
     a1 = getattr(class1, key)
     a2 = getattr(class2, key)
 
-    assert type(a1) == type(a2)
+    assert type(a1) == type(a2)  # noqa
 
     if isinstance(a1, str) and allow_str_case_diff:
         assert a1.upper() == a2.upper()