Add giant analyze metrics plots for training progress (gitea #75) #78

Merged

lars merged 3 commits from fix/issue-75 into master 2026-08-24 11:32:35 +02:00

Conversation 0 Commits 3 Files Changed 3

+708

lars commented 2026-08-24 10:56:45 +02:00

Owner

Copy Link

Copy Source

MetricsCollector writes one row per epoch to <run_dir>/metrics.csv, but
nothing read or plotted it. giant/training/plots.py reads the CSV header
dynamically (the column set varies by run: flow/ddpm vs wgan, routed vs
not) and renders loss/lr/accuracy/grad-norm/router/wgan-balance/throughput
plots with the same plotstyle conventions giant/analysis/render.py uses,
skipping any figure whose columns aren't present for a given run.

Wired up as giant analyze metrics <run_dir>, writing PDFs into the same
gitignored analysis_runs/ directory analyze prep/submit already use
(derive_metrics_dir mirrors derive_run_dir) rather than into the training
run directory itself.

MetricsCollector writes one row per epoch to <run_dir>/metrics.csv, but nothing read or plotted it. giant/training/plots.py reads the CSV header dynamically (the column set varies by run: flow/ddpm vs wgan, routed vs not) and renders loss/lr/accuracy/grad-norm/router/wgan-balance/throughput plots with the same plotstyle conventions giant/analysis/render.py uses, skipping any figure whose columns aren't present for a given run. Wired up as `giant analyze metrics <run_dir>`, writing PDFs into the same gitignored analysis_runs/ directory `analyze prep`/`submit` already use (derive_metrics_dir mirrors derive_run_dir) rather than into the training run directory itself.

lars added 1 commit 2026-08-24 10:56:46 +02:00

Add giant analyze metrics plots for training progress (gitea #75) ... bdebd83c8b

MetricsCollector writes one row per epoch to <run_dir>/metrics.csv, but
nothing read or plotted it. giant/training/plots.py reads the CSV header
dynamically (the column set varies by run: flow/ddpm vs wgan, routed vs
not) and renders loss/lr/accuracy/grad-norm/router/wgan-balance/throughput
plots with the same plotstyle conventions giant/analysis/render.py uses,
skipping any figure whose columns aren't present for a given run.

Wired up as `giant analyze metrics <run_dir>`, writing PDFs into the same
gitignored analysis_runs/ directory `analyze prep`/`submit` already use
(derive_metrics_dir mirrors derive_run_dir) rather than into the training
run directory itself.

lars added 1 commit 2026-08-24 11:15:59 +02:00

Fix LaTeX-unavailable skip check in analyze metrics smoke test ... 7560e2bff0

CliRunner stores an uncaught exception in result.exception, not
result.output, so the skip condition never matched and the test
failed outright on CI machines without LaTeX installed.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

lars added 1 commit 2026-08-24 11:32:23 +02:00

Merge branch 'master' into fix/issue-75 37d73e6578

lars merged commit 417b741484 into master 2026-08-24 11:32:35 +02:00

lars deleted branch fix/issue-75 2026-08-24 11:32:35 +02:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

architecture

bug

Silent/wrong behavior, no crash

experiment

feature

New feature matching the v0.3 redesign

investigation

modularity

Making the network genuinely pluggable

performance

phase2

validation

wontfix

Not being fixed right now

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

lars (Lars Bogner)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: lars/giant#78