Skip to content

Conversation

@aperov9
Copy link

@aperov9 aperov9 commented Nov 17, 2025

This pull request introduces test coverage reporting for the Python-based code base & uploads the merged coverage to codecov.

CI/CD and Coverage Reporting Enhancements

  • Updated the Python workflow to install the coverage package, run tests with coverage enabled for all test modes (including federated and scuro), and collect per-mode coverage data files. Added a new job to merge these coverage files, generate a combined report, and upload it to Codecov and as workflow artifacts.
  • Modified the Java test workflow to tag Codecov uploads with a java flag for clearer separation of coverage data.
  • Please make sure to enable the flag feature in codecov.

Documentation and Badge Updates

  • Updated the README.md to display separate Codecov badges for Java and Python coverage.

Python Coverage Configuration

  • Added a .coveragerc configuration file in src/main/python to omit irrelevant files (such as those generated by OpenCV and PyTorch) from coverage analysis.

@codecov
Copy link

codecov bot commented Nov 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.23%. Comparing base (b394e32) to head (93767c4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #2359       +/-   ##
=============================================
+ Coverage     71.51%   83.23%   +11.72%     
=============================================
  Files          1539      330     -1209     
  Lines        182605     9283   -173322     
  Branches      35916        0    -35916     
=============================================
- Hits         130585     7727   -122858     
+ Misses        42028     1556    -40472     
+ Partials       9992        0     -9992     
Flag Coverage Δ
python 83.23% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aperov9 aperov9 marked this pull request as ready for review November 20, 2025 14:24
@github-project-automation github-project-automation bot moved this from In Progress to Done in SystemDS PR Queue Jan 9, 2026
@christinadionysio
Copy link
Contributor

@aperov9 could you please merge main into this commit so I can trigger a rerun of the codecov action?

@aperov9
Copy link
Author

aperov9 commented Jan 11, 2026

Done @christinadionysio

@aperov9
Copy link
Author

aperov9 commented Jan 12, 2026

Hmm, seems like something doesn't work anymore for the python test, will check that. However, the java tests should run, I did not touch anything there. Can you confirm that the tests are just flaky or that is broken on main? @christinadionysio

@christinadionysio
Copy link
Contributor

Hmm, seems like something doesn't work anymore for the python test, will check that. However, the java tests should run, I did not touch anything there. Can you confirm that the tests are just flaky or that is broken on main? @christinadionysio

@aperov9 the java tests are running now, would be great if you could look into the failing python tests.

@aperov9
Copy link
Author

aperov9 commented Jan 24, 2026

@christinadionysio, sorry for the delay. Can you please re-run it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants