github/ai-toolkit
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Enable val/loss by default on the loss graph (#966)

Browse Source
This commit is contained in:
fatalis
2026-07-20 12:33:03 -05:00
committed by GitHub
parent 088084e2c2
commit c2864bba48
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
ui/src/components/JobLossGraph.tsx
View File

@@ -212,8 +212,9 @@ export default function JobLossGraph({ job }: Props) {
}
}, [hydrated, useLogScale, showTrend, smoothing, plotStride, clipOutliers, enabled]);
// keep enabled map in sync with discovered keys. Only "loss/loss" is on by
// default; every other metric starts deactivated (user can toggle it on).
// keep enabled map in sync with discovered keys. "loss/loss" and "val/loss"
// are on by default; every other metric starts deactivated (user can toggle
// it on).
useEffect(() => {
// Nothing discovered yet — don't prune, or we'd wipe a restored selection
// before the keys have loaded.
@@ -221,7 +222,7 @@ export default function JobLossGraph({ job }: Props) {
setEnabled(prev => {
const next = { ...prev };
for (const k of lossKeys) {
if (next[k] === undefined) next[k] = persistedEnabledRef.current?.[k] ?? k === 'loss/loss';
if (next[k] === undefined) next[k] = persistedEnabledRef.current?.[k] ?? (k === 'loss/loss' || k === 'val/loss');
}
for (const k of Object.keys(next)) {
if (!lossKeys.includes(k)) delete next[k];