Update flake8 plugin versions to match CI

other minor cleanup
RadioAstronomySoftwareGroup · Dec 7, 2023 · abd6408 · abd6408
1 parent c643bfd
commit abd6408
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 13 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -20,14 +20,14 @@ repos:
     hooks:
       - id: flake8
         additional_dependencies:
-          - flake8-bugbear>=23.1.17
-          - flake8-builtins
-          - flake8-comprehensions
-          - flake8-docstrings
-          # - flake8-eradicate  # not currently compatible with others
-          - flake8-pytest-style
-          - flake8-rst-docstrings
-          - pep8-naming
+          - flake8-bugbear>=23.12.2
+          - flake8-builtins>=2.2.0
+          - flake8-comprehensions>=3.14.0
+          - flake8-docstrings>=1.7.0
+          # - flake8-eradicate>=1.5.0  # many false-positives
+          - flake8-pytest-style>=1.7.2
+          - flake8-rst-docstrings>=0.3.0
+          - pep8-naming>=0.13.3
 
   - repo: https://github.com/psf/black
     rev: 23.11.0

diff --git a/pyuvdata/uvbeam/tests/test_uvbeam.py b/pyuvdata/uvbeam/tests/test_uvbeam.py
@@ -2854,11 +2854,9 @@ def test_beam_area_healpix(
     assert np.allclose(xx_area, xx_area)
     yy_area = healpix_norm.get_beam_area("YY")
     assert np.allclose(yy_area / xx_area, np.ones(numfreqs))
-    # nt.assert_almost_equal(yy_area / xx_area, 1.0, places=5)
     xx_area = healpix_norm.get_beam_sq_area("XX")
     yy_area = healpix_norm.get_beam_sq_area("YY")
     assert np.allclose(yy_area / xx_area, np.ones(numfreqs))
-    # nt.assert_almost_equal(yy_area / xx_area, 1.0, places=5)
 
     # Check that if pseudo-Stokes I (pI) is in the beam polarization_array it
     # just uses it

diff --git a/pyuvdata/uvdata/mwa_corr_fits.py b/pyuvdata/uvdata/mwa_corr_fits.py
@@ -1023,7 +1023,6 @@ def _flag_small_auto_ants(
         # look for small autos and flag
         auto_inds = self.ant_1_array == self.ant_2_array
         autos = self.data_array.real[auto_inds, :, 0:2]
-        # auto_flags = self.flag_array[auto_inds, :, 0:2]
         autos = autos.reshape(self.Ntimes, self.Nants_data, self.Nfreqs, 2)
         # find autos below threshold
         small_auto_flags = np.logical_and(autos != 0, autos <= threshold)

diff --git a/pyuvdata/uvdata/uvdata.py b/pyuvdata/uvdata/uvdata.py
@@ -1814,8 +1814,7 @@ def print_phase_center_info(
 
         dict_list = [self.phase_center_catalog[cat_id] for cat_id in cat_id_list]
 
-        # We want to check and actually see which fields we need to
-        # print
+        # We want to check and actually see which fields we need to print
         any_lon = any_lat = any_frame = any_epoch = any_times = False
         any_pm_ra = any_pm_dec = any_dist = any_vrad = False