Checkpoint loading can mix model weights and training state from different epochs
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Área
- machine-learning
Línea de trabajo
Start at the load_checkpoint and save_checkpoint entry points and run the minimal reproduction to observe the mismatched epoch, model weight, and optimizer state. Ensure loading chooses one checkpoint index for all requested models and training state, fails before mutation when a required file is missing, and preserves model-only directory behavior; verify the single-process and distributed cases described in the issue.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Version and installation
Source checkout: main at 94dbdf829d1a4e93e3f31ecb77713392c471388e. Also reproduced in the original #2010 implementation at 01757c816713892c38455d489376494ee7ee11e5. Python 3.13.8, PyTorch 2.12.0+cu130, Linux.
Description
load_checkpoint(..., epoch=None) independently selects the latest surviving file for the training state and for each requested model. If the newest model weights are deleted but an older weights file remains, loading succeeds with old model weights and newer optimizer/scheduler state. It reports the newer epoch, so training silently resumes from an inconsistent combination of states.
The same mismatch can occur in the other direction: a newer model file with no matching training-state file is combined with older training state.
Minimal reproduction
from pathlib import Path
from tempfile import TemporaryDirectory
import torch
from physicsnemo.utils import load_checkpoint, save_checkpoint
with TemporaryDirectory() as directory:
path = Path(directory)
model = torch.nn.Linear(1, 1, bias=False)
optimizer = torch.optim.Adam(model.parameters(), lr=0.01)
for epoch in (1, 2):
with torch.no_grad():
model.weight.fill_(epoch)
optimizer.param_groups[0]["lr"] = epoch * 0.01
save_checkpoint(path, models=model, optimizer=optimizer, epoch=epoch)
(path / "Linear.0.2.pt").unlink()
fresh = torch.nn.Linear(1, 1, bias=False)
fresh_optimizer = torch.optim.Adam(fresh.parameters(), lr=0.5)
epoch = load_checkpoint(path, models=fresh, optimizer=fresh_optimizer)
print(epoch, fresh.weight.item(), fresh_optimizer.param_groups[0]["lr"])
Observed output:
2 1.0 0.02
The returned epoch and optimizer learning rate come from epoch 2; the model weight comes from epoch 1. No exception is raised.
Expected behavior
Select one training checkpoint index and require every requested model's weights at that same index. If any required file is missing, fail clearly before changing model or training state. The caller can explicitly select an older complete checkpoint. Do not independently fall back to older or newer model files.
This should also work for automatically numbered saves, where the filename index may exist without an epoch key in the training-state payload. Preserve current behavior for directories containing only model weights.
Scope and verification
Reproduced with single-process loading and on both ranks of a two-process CPU/Gloo DTensor run, including optimizer and scheduler restoration. The filename selection is independent of the device backend. The issue exists on main and was identified while reviewing #2010; that PR will address it.
- Lenguaje dominante
- Python
- Estrellas
- 3.3k
- Forks
- 787
- Merge medio
- 2 d 19 h
- PR fusionados (30 d)
- 24
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de NVIDIA/physicsnemo
-
? - Needs Triage bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
NVIDIA/physicsnemo#2021 ·
Los mantenedores suelen responder en 1 día
-
? - Needs Triage bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
NVIDIA/physicsnemo#2020 ·
Los mantenedores suelen responder en 1 día
-
🐛[BUG]: power_spectrum in metrics.general puts the zero frequency half a cell off along an odd widthAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
NVIDIA/physicsnemo#2007 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
NVIDIA/physicsnemo#2005 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
NVIDIA/physicsnemo#2003 ·
Los mantenedores suelen responder en 1 día
Todos los issues de NVIDIA/physicsnemo
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
https://search.utilibre.orgAbiertoinstance instance add
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
searxng/searx-instances#941 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
FluidNumerics/fluid-walk-blocker#89 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día