Fix for new unit test data#1008
Conversation
|
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
sarthakpati
left a comment
There was a problem hiding this comment.
LGTM
Pending confirmation from @noopurintel before merging.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1008 +/- ##
=======================================
Coverage 82.12% 82.12%
=======================================
Files 193 193
Lines 12290 12291 +1
=======================================
+ Hits 10093 10094 +1
Misses 2197 2197
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
It worked fine - https://github.com/noopurintel/openfl/actions/runs/14920993167/job/41916194525#step:6:4 |
Thanks for the confirming! Once your comment has been addressed, we can merge. |
Fixes #1007
Proposed Changes
Checklist
CONTRIBUTINGguide has been followed.typingis used to provide type hints, including and not limited to usingOptionalif a variable has a pre-defined value).pip installstep is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].logginglibrary is being used and noprintstatements are left.