Skip to content

Add tests for aggregator checks#264

Merged
ValerianRey merged 6 commits intomainfrom
add-aggregator-tests
Mar 23, 2025
Merged

Add tests for aggregator checks#264
ValerianRey merged 6 commits intomainfrom
add-aggregator-tests

Conversation

@ValerianRey
Copy link
Copy Markdown
Contributor

@ValerianRey ValerianRey commented Mar 23, 2025

  • Add missing TrimmedMean tests
  • Add missing Krum tests
  • Add missing Constant tests
  • Add missing CAGrad tests
  • Add missing GradDrop tests
  • Add missing (base) Aggregator test

=> +1% code coverage 🥳

@ValerianRey ValerianRey added cc: test Conventional commit type for changes to tests. package: aggregation labels Mar 23, 2025
@ValerianRey ValerianRey self-assigned this Mar 23, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

see 6 files with indirect coverage changes

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

@ValerianRey ValerianRey marked this pull request as ready for review March 23, 2025 10:57
@ValerianRey ValerianRey requested review from PierreQuinton and removed request for PierreQuinton March 23, 2025 10:58
@ValerianRey
Copy link
Copy Markdown
Contributor Author

@PierreQuinton FYI

@ValerianRey ValerianRey merged commit 480705b into main Mar 23, 2025
14 checks passed
@ValerianRey ValerianRey deleted the add-aggregator-tests branch March 23, 2025 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cc: test Conventional commit type for changes to tests. package: aggregation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant