Skip to content

Add tests for interpolate_diskarray function#67

Open
felixcremer wants to merge 8 commits into
mainfrom
fc/aggregate
Open

Add tests for interpolate_diskarray function#67
felixcremer wants to merge 8 commits into
mainfrom
fc/aggregate

Conversation

@felixcremer
Copy link
Copy Markdown
Collaborator

This also adds tests for aggregate and some printing changes.

@meggart I haven't done the fully inner test yet, but the other two are in the test_interpolation file.

@coveralls
Copy link
Copy Markdown

coveralls commented May 18, 2026

Coverage Report for CI Build 26103391243

Coverage increased (+12.4%) to 58.111%

Details

  • Coverage increased (+12.4%) from the base build.
  • Patch coverage: 9 uncovered changes across 2 files (0 of 9 lines covered, 0.0%).
  • 1 coverage regression across 1 file.

Uncovered Changes

File Changed Covered %
src/mwops.jl 6 0 0.0%
src/diskarrayresults.jl 3 0 0.0%

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
src/mwops.jl 1 51.55%

Coverage Stats

Coverage Status
Relevant Lines: 2139
Covered Lines: 1243
Line Coverage: 58.11%
Coverage Strength: 211.77 hits per line

💛 - Coveralls

@codecov
Copy link
Copy Markdown

codecov Bot commented May 18, 2026

Codecov Report

❌ Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.11%. Comparing base (4808ae7) to head (741f5fe).

Files with missing lines Patch % Lines
src/mwops.jl 0.00% 6 Missing ⚠️
src/diskarrayresults.jl 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #67       +/-   ##
===========================================
+ Coverage   45.75%   58.11%   +12.35%     
===========================================
  Files          23       23               
  Lines        2133     2139        +6     
===========================================
+ Hits          976     1243      +267     
+ Misses       1157      896      -261     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@meggart
Copy link
Copy Markdown
Owner

meggart commented May 18, 2026

Nice, thank you! Any idea why the tests are not failing?

@felixcremer
Copy link
Copy Markdown
Collaborator Author

CI is not failing, because I forgot to include the test_interpolation file

@felixcremer
Copy link
Copy Markdown
Collaborator Author

I included test_interpolation.jl into runtests.jl and fixed the tests. These are now succeding because I commented the range dispatch of getinterpinds. We could also just leave it like that for now and look at the range method for getinterpinds if this is really the bottleneck for some application.

@felixcremer
Copy link
Copy Markdown
Collaborator Author

This is missing tests for the show functions for GMWOPResults and LoopWindows

@meggart
Copy link
Copy Markdown
Owner

meggart commented May 19, 2026

Yes let's leave this as it is right now. One or 2 tests for the new print methods would be great, even if only skeleton. I will look into extending the GMWOPResult show method tomorrow.

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