Skip to content

Add NC validation metrics and fix mini-batch local updates - #45

Merged
yh-yao merged 3 commits into
mainfrom
update_hc
Jul 16, 2026
Merged

Add NC validation metrics and fix mini-batch local updates#45
yh-yao merged 3 commits into
mainfrom
update_hc

Conversation

@Pulserius

Copy link
Copy Markdown
Contributor

Summary

  • Propagate validation splits through the NC data-loading and trainer paths.
  • Report weighted global validation loss and accuracy during NC training, while keeping test evaluation final-only.
  • Correct mini-batch NC training so each local step consumes one seed batch and performs one model update.
  • Preserve compatibility with legacy NC data tuples that do not include validation fields.

Scope

Benchmark scripts, benchmark results, elastic-round experiments, and KubeRay/Prometheus work are intentionally excluded. They remain on update_hc_experiments.

Validation

  • Compiled the changed NC modules with Python 3.12.
  • Ran the targeted validation, data-path, trainer, mini-batch, and NC integration tests: 15 passed.
  • The repository has unrelated pre-existing failures in broader unit suites; the equivalent federated-method failures are also present on main.

@Pulserius
Pulserius requested review from cyfan11 and yh-yao July 14, 2026 20:27
@yh-yao
yh-yao merged commit b7b073e into main Jul 16, 2026
2 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.

2 participants