Skip to content

Conversation

@lukehoffmann
Copy link
Contributor

Working on test coverage for the pyearthtools.pipeline.operations.numpy namespace.

This is the coverage before I started:

File coverage
pyearthtools/pipeline/operations/numpy/init.py 100%
pyearthtools/pipeline/operations/numpy/augment.py 35%
pyearthtools/pipeline/operations/numpy/conversion.py 100%
pyearthtools/pipeline/operations/numpy/filters.py 33%
pyearthtools/pipeline/operations/numpy/join.py 60%
pyearthtools/pipeline/operations/numpy/normalisation.py 45%
pyearthtools/pipeline/operations/numpy/reshape.py 89%
pyearthtools/pipeline/operations/numpy/select.py 25%
pyearthtools/pipeline/operations/numpy/split.py 41%
pyearthtools/pipeline/operations/numpy/values.py 42%

(Also currently contains a couple of tests for pyearthtools.data.transforms.derive that I started on in the same branch)

@lukehoffmann lukehoffmann changed the title Coverage Test coverage for pyearthtools.pipeline.operations.numpy Nov 14, 2025
@coveralls
Copy link

coveralls commented Nov 14, 2025

Pull Request Test Coverage Report for Build 19354487537

Details

  • 90 of 90 (100.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 63.212%

Totals Coverage Status
Change from base Build 19354196491: 0.4%
Covered Lines: 10043
Relevant Lines: 15494

💛 - Coveralls

@tennlee tennlee marked this pull request as ready for review November 19, 2025 13:38
@tennlee
Copy link
Collaborator

tennlee commented Nov 19, 2025

This looks really helpful, thank you. I see it's still marked as draft, but since the hackathon is finished and the work has reached a useful level, I'm going to merge it and you're welcome to create a new PR for any continuation of the work. Thanks very much!

@tennlee tennlee merged commit 005aff4 into ACCESS-Community-Hub:develop Nov 19, 2025
5 of 6 checks passed
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.

3 participants