spacetelescope · WilliamJamieson · Feb 5, 2024 · Feb 5, 2024 · Feb 5, 2024
@@ -32,7 +32,7 @@ repos:
       args: ["--py38-plus"]
 
 - repo: https://github.com/astral-sh/ruff-pre-commit
-  rev: 'v0.1.13'
+  rev: 'v0.2.0'
   hooks:
     - id: ruff
       args: ["--fix"]
@@ -43,12 +43,12 @@ repos:
     - id: isort
 
 - repo: https://github.com/psf/black
-  rev: 23.12.1
+  rev: 24.1.1
   hooks:
     - id: black
 
 - repo: https://github.com/PyCQA/bandit
-  rev: 1.7.6
+  rev: 1.7.7
   hooks:
     - id: bandit
       args: ["-c", "pyproject.toml"]

@@ -2,6 +2,7 @@
 Assign a gWCS object to a science image.
 
 """
+
 import logging
 
 import gwcs.coordinate_frames as cf

@@ -7,6 +7,7 @@
 For more, see the :ref:`documentation overview <asn-overview>`.
 
 """
+
 # flake8: noqa: F401, F402, F403
 
 # Take version from the upstream package

@@ -1,4 +1,5 @@
 """Create an association from a list"""
+
 import argparse
 import sys
 

@@ -1,6 +1,7 @@
 """
 Define the I/O methods for Level 3 associations
 """
+
 import json as json_lib
 import logging
 

@@ -1,4 +1,5 @@
 """Association Candidate Identifier"""
+
 import re
 from ast import literal_eval
 

@@ -4,6 +4,7 @@
 -----
 These associations are specifically defined for use in DMS.
 """
+
 from romancal.associations.lib import rules_level2
 from romancal.associations.registry import RegistryMarker
 

@@ -1,5 +1,6 @@
 """Constraints
 """
+
 import abc
 import collections
 import logging

@@ -7,6 +7,7 @@
 
 .. _glue-viz: https://github.com/glue-viz/glue
 """
+
 from functools import wraps
 
 __all__ = ["memoize", "singleton", "memoize_attr_check"]

@@ -1,4 +1,5 @@
 """Association Member"""
+
 from collections import UserDict
 from copy import copy
 

@@ -1,4 +1,5 @@
 """Reprocessing List"""
+
 from collections import deque
 from enum import Enum
 from functools import reduce

@@ -1,5 +1,6 @@
 """Association Definitions: DMS Level2b product associations
 """
+
 import logging
 
 from romancal.associations.lib.constraint import Constraint

@@ -1,4 +1,5 @@
 """Update path of members in an association"""
+
 from os.path import basename, join
 
 

@@ -1,4 +1,5 @@
 """General Utilities"""
+
 import logging
 from ast import literal_eval
 from functools import wraps

@@ -1,4 +1,5 @@
 """Load an Association from a file or object"""
+
 from inspect import isclass
 
 from .association import Association

@@ -1,4 +1,5 @@
 """Main entry for the association generator"""
+
 import argparse
 import logging
 import os

@@ -1,4 +1,5 @@
 """Association Registry"""
+
 import importlib.util
 import logging
 from inspect import getmembers, isclass, isfunction, ismethod, ismodule

@@ -1,4 +1,5 @@
 """Helpers for tests."""
+
 # ruff: noqa
 import os
 import re

@@ -1,4 +1,5 @@
 """Constraint Tests"""
+
 import pytest
 
 from romancal.associations.lib.constraint import (

@@ -1,4 +1,5 @@
 """Test Level2 candidate operation"""
+
 import pytest
 
 from romancal.associations.main import Main

@@ -1,4 +1,5 @@
 """Test the Registry"""
+
 import pytest
 
 from romancal.associations.lib.keyvalue_registry import KeyValueRegistry

@@ -1,4 +1,5 @@
 """Test utility update_path"""
+
 from romancal.associations.asn_from_list import asn_from_list
 from romancal.associations.lib.rules_elpp_base import DMS_ELPP_Base
 from romancal.associations.lib.update_path import update_path

@@ -1,4 +1,5 @@
 """Project default for pytest"""
+
 import inspect
 import json
 import os

@@ -44,9 +44,9 @@
             if "nframes" not in dark_model.meta.exposure:
                 dark_model.meta.exposure["nframes"] = input_model.meta.exposure.nframes
             if "groupgap" not in dark_model.meta.exposure:
-                dark_model.meta.exposure[
-                    "groupgap"
-                ] = input_model.meta.exposure.groupgap
+                dark_model.meta.exposure["groupgap"] = (
+                    input_model.meta.exposure.groupgap
+                )
 
             # Do the dark correction
             out_model = input_model

@@ -2,7 +2,6 @@
 Unit tests for dark current correction
 """
 
-
 import numpy as np
 import pytest
 import roman_datamodels as rdm

@@ -198,9 +198,11 @@ def __getitem__(self, index):
             step = index.step
             m = self._models[start:stop:step]
             m = [
-                rdm.open(item, memmap=self._memmap)
-                if (not isinstance(item, rdm.DataModel) and self._return_open)
-                else item
+                (
+                    rdm.open(item, memmap=self._memmap)
+                    if (not isinstance(item, rdm.DataModel) and self._return_open)
+                    else item
+                )
                 for item in m
             ]
         else:

@@ -1,6 +1,7 @@
 """
  Unit tests for the Roman jump step code
 """
+
 from itertools import cycle
 
 import numpy as np

@@ -3,6 +3,7 @@
 Example: Certain tests are required by DMS to log in a specific way. The
 decorator for this is defined in this module.
 """
+
 from stpipe import log as stpipe_log
 
 

@@ -18,6 +18,7 @@
 which provides 32 bits. Bits of an integer are most easily referred to using
 the formula `2**bit_number` where `bit_number` is the 0-index bit of interest.
 """
+
 # for this file drop E241 (multiple spaces after :, for readability)
 # flake8: noqa: E241
 # Pixel-specific flags

@@ -6,6 +6,7 @@
 
 If the module is not available, then stub it out.
 """
+
 import logging
 
 __all__ = ["Bar"]

@@ -1,5 +1,6 @@
 """ A signal/slot implementation
 """
+
 import inspect
 import logging
 from collections import namedtuple

@@ -20,6 +20,7 @@
 `SUFFIXES_TO_DISCARD` as necessary, then use the output of
 `find_suffixes`.
 """
+
 import itertools
 import logging
 import re

@@ -3,6 +3,7 @@
 Unit tests for linearity correction
 
 """
+
 import numpy as np
 import pytest
 from astropy import units as u

@@ -2,6 +2,7 @@
 This module collects all of the stpipe.Pipeline subclasses
 made available by this package.
 """
+
 from .exposure_pipeline import ExposurePipeline
 from .highlevel_pipeline import HighLevelPipeline
 

@@ -20,7 +20,6 @@
 
 
 class RampFitStep(RomanStep):
-
     """
     This step fits a straight line to the value of counts vs. time to
     determine the mean count rate for each pixel.
@@ -262,9 +261,7 @@ def create_image_model(input_model, image_info):
     var_poisson = u.Quantity(
         var_poisson, u.electron**2 / u.s**2, dtype=var_poisson.dtype
     )
-    var_rnoise = u.Quantity(
-        var_rnoise, u.electron**2 / u.s**2, dtype=var_rnoise.dtype
-    )
+    var_rnoise = u.Quantity(var_rnoise, u.electron**2 / u.s**2, dtype=var_rnoise.dtype)
     err = u.Quantity(err, u.electron / u.s, dtype=err.dtype)
     if dq is None:
         dq = np.zeros(data.shape, dtype="u4")

@@ -466,9 +466,9 @@ def fft_interpolate(self, num: int = 3) -> ChannelView:
 
         # Mask all the data columns
         mask = np.ones((rows, columns), dtype=bool)
-        mask[
-            :, : -Const.PAD
-        ] = False  # this maybe different for the left/right channels
+        mask[:, : -Const.PAD] = (
+            False  # this maybe different for the left/right channels
+        )
         mask = mask.flatten()
 
         # Find the indices of the padded columns

@@ -14,6 +14,7 @@
 provided reference code and this implementation contained within the git history.
 of the project itself.
 """
+
 import threading
 
 import numpy as np
@@ -308,7 +309,7 @@ def fft_interp(
             # meaningfully
             chanFrameData_Flat = spfft.irfft(
                 fftResult * chanFrameData_Flat.size,
-                workers=1
+                workers=1,
                 # )
             ).astype(chanFrameData_Flat.dtype)
             # Return read only pixels

@@ -726,9 +726,10 @@ def compare_asdf(result, truth, ignore=None, rtol=1e-05, atol=1e-08, equal_nan=T
         "lazy_load": False,
         "copy_arrays": True,
     }
-    with asdf.open(result, **open_kwargs) as af0, asdf.open(
-        truth, **open_kwargs
-    ) as af1:
+    with (
+        asdf.open(result, **open_kwargs) as af0,
+        asdf.open(truth, **open_kwargs) as af1,
+    ):
         # swap the inputs here so DeepDiff(truth, result)
         # this will create output with 'new_value' referring to
         # the value in the result and 'old_value' referring to the truth

@@ -1,4 +1,5 @@
 """Regression test for the Dark current subtraction step."""
+
 import pytest
 
 from romancal.stpipe import RomanStep

@@ -1,4 +1,5 @@
 """Regression test for the Jump detection step."""
+
 import pytest
 
 from romancal.stpipe import RomanStep

@@ -1,4 +1,5 @@
 """Regression test for the linearity correction step."""
+
 import pytest
 
 from romancal.stpipe import RomanStep

@@ -5,6 +5,7 @@
 A requirement for the larger mission verification project is to have
 tests tied to reqirements.
 """
+
 from pathlib import Path
 
 import pytest

@@ -1,4 +1,5 @@
 """Regression tests for the Reference Pixel Correction step"""
+
 import pytest
 
 from romancal.stpipe import RomanStep

@@ -1,4 +1,5 @@
 """Tests for the DQ Init module and DMS 25 and DMS 26 requirements"""
+
 import os
 
 import pytest

@@ -1,4 +1,5 @@
 """Regression tests for the photom step of the Roman pipeline"""
+
 import math
 
 import pytest

@@ -1,4 +1,5 @@
 """ Roman tests for flat field correction """
+
 import copy
 
 import numpy as np

@@ -1,4 +1,5 @@
 """ Tests for the saturation step"""
+
 import os
 
 import pytest

@@ -1,6 +1,7 @@
 """
 A module that provides functions for matching sky in overlapping images.
 """
+
 import logging
 from datetime import datetime
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,6 +2,7 @@ @@
     Assign a gWCS object to a science image.
     """
     import logging
     import gwcs.coordinate_frames as cf
@@ Expand Down @@
Original file line number	Diff line number	Diff line change
		@@ -1,4 +1,5 @@
		"""Update path of members in an association"""

		from os.path import basename, join


Expand Down