Skip to content

Comments

Apply a good chunk of linting#149

Merged
tennlee merged 5 commits intodevelopfrom
linting
Jul 29, 2025
Merged

Apply a good chunk of linting#149
tennlee merged 5 commits intodevelopfrom
linting

Conversation

@tennlee
Copy link
Collaborator

@tennlee tennlee commented Jul 29, 2025

Brings the utils subpackage into compliance with ruff check
Partially improves the packages subpackage

tennlee added 5 commits July 29, 2025 19:03
Applied automatically with "ruff check --fix"
Automated tests pass
Diffs reviewed
Improve compliance for static analysis
Manually review all changes
Automated tests pass
@coveralls
Copy link

Pull Request Test Coverage Report for Build 16593476845

Details

  • 27 of 47 (57.45%) changed or added relevant lines in 29 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.005%) to 60.448%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/pipeline/src/pyearthtools/pipeline/operations/numpy/split.py 0 1 0.0%
packages/utils/src/pyearthtools/utils/data/tesselator/_patching/patches.py 2 3 66.67%
packages/utils/src/pyearthtools/utils/parameter.py 0 1 0.0%
packages/pipeline/src/pyearthtools/pipeline/controller.py 0 2 0.0%
packages/pipeline/src/pyearthtools/pipeline/operations/dask/filters.py 0 2 0.0%
packages/pipeline/src/pyearthtools/pipeline/operations/numpy/filters.py 0 2 0.0%
packages/pipeline/src/pyearthtools/pipeline/operations/numpy/reshape.py 1 3 33.33%
packages/pipeline/src/pyearthtools/pipeline/operations/xarray/filters.py 0 2 0.0%
packages/utils/src/pyearthtools/utils/initialisation/imports.py 0 2 0.0%
packages/pipeline/src/pyearthtools/pipeline/operations/dask/reshape.py 0 5 0.0%
Files with Coverage Reduction New Missed Lines %
packages/pipeline/src/pyearthtools/pipeline/operations/dask/reshape.py 1 25.41%
Totals Coverage Status
Change from base Build 16591380478: 0.005%
Covered Lines: 10131
Relevant Lines: 16258

💛 - Coveralls

@tennlee tennlee merged commit fe2019c into develop Jul 29, 2025
15 checks passed
@tennlee tennlee deleted the linting branch July 29, 2025 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants