Skip to content

Commit

Permalink
[pre-commit.ci] auto fixes from pre-commit.com hooks
Browse files Browse the repository at this point in the history
for more information, see https://pre-commit.ci
  • Loading branch information
pre-commit-ci[bot] committed Sep 19, 2024
1 parent 3324f5a commit 120c057
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions gpm/utils/collocation.py
Original file line number Diff line number Diff line change
Expand Up @@ -125,8 +125,8 @@ def collocate_product(

# Concatenate if necessary (PMW case)
output_ds = xr.concat(list_remapped, dim="pmw_frequency") if len(list_remapped) > 1 else list_remapped[0]
# Add time of dst dataset
output_ds = output_ds.assign_coords({"time" : ds.reset_coords()["time"]})
# Add time of dst dataset
output_ds = output_ds.assign_coords({"time": ds.reset_coords()["time"]})

Check notice on line 129 in gpm/utils/collocation.py

View check run for this annotation

CodeScene Delta Analysis / CodeScene Cloud Delta Analysis (main)

ℹ Getting worse: Excess Number of Function Arguments

collocate_product increases from 10 to 11 arguments, threshold = 4. This function has too many arguments, indicating a lack of encapsulation. Avoid adding more arguments.
# Assign attributes
output_ds.attrs = list_ds[0].attrs
output_ds.attrs["ScanMode"] = scan_modes
Expand Down

0 comments on commit 120c057

Please sign in to comment.