Repository navigation
Add optional diffusion distillation strategy - #2099
Closed
CharlelieLrt wants to merge 1 commit into
Closed
CharlelieLrt wants to merge 1 commit into
CharlelieLrt wants to merge 1 commit into
Conversation
Contributor
CODEOWNERS review mapCurrent for commit ⏳ @CharlelieLrt — 7 file(s)
⏳ @coreyjadams — 1 file(s)
⏳ @ktangsali — 1 file(s)
⏳ @megnvidia — 1 file(s)
No CODEOWNER
Comment |
CharlelieLrt
force-pushed
the
codex/diffusion-distillation-strategy
branch
from
October 14, 2026 05:35
bc16396 to
b8c31db
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-checkpoint-bundle
branch
from
October 14, 2026 05:35
b19f3dc to
251d5db
Compare
This was referenced Oct 14, 2026
CharlelieLrt
force-pushed
the
codex/diffusion-checkpoint-bundle
branch
from
October 14, 2026 17:03
251d5db to
6fb97f0
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-distillation-strategy
branch
from
October 14, 2026 17:03
b8c31db to
859b1c8
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-checkpoint-bundle
branch
from
October 16, 2026 01:19
6fb97f0 to
7a29c5f
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-distillation-strategy
branch
2 times, most recently
from
October 16, 2026 02:52
71c4445 to
b7f4957
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-checkpoint-bundle
branch
2 times, most recently
from
October 16, 2026 20:15
8b9bbd5 to
62a1dba
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-distillation-strategy
branch
2 times, most recently
from
October 17, 2026 01:44
7ac3370 to
5ed9710
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-checkpoint-bundle
branch
from
October 17, 2026 01:44
62a1dba to
6a7ad14
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-distillation-strategy
branch
from
October 28, 2026 03:42
5ed9710 to
f7ae646
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-checkpoint-bundle
branch
from
October 28, 2026 03:42
6a7ad14 to
76416bf
Compare
Signed-off-by: Charlelie Laurent <claurent@nvidia.com>
CharlelieLrt
force-pushed
the
codex/diffusion-checkpoint-bundle
branch
from
October 29, 2026 02:23
76416bf to
d775406
Compare
CharlelieLrt
force-pushed
the
codex/diffusion-distillation-strategy
branch
from
October 29, 2026 02:23
f7ae646 to
78c4e28
Compare
This was referenced Oct 29, 2026
Collaborator
Author
|
Continued in PhysicsNeMo #2204, from branch |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds the optional
DistillationStrategyfor users who do not need towrite their own distillation loop. It extends Alchemi's general training strategy
to manage phase-specific optimizers, gradient accumulation, AMP, EMA, and
checkpoint restart.
Users install the
diffusion-strategyextra and supply public models, a nativeor FastGen-backed loss, and TensorDict batches:
For explicit MeanFlow times, store
tandras separate batch tensors.Use
training_fn(models, batch)to return loss arguments containing(t, DiffusionExtraTimes(r=r, batch_size=[B])). Ordinary single-time batchesneed no conversion.
The strategy remains optional: expert users can call the same loss from their
own loop. Compilation and distributed setup remain under user control.
This PR also carries the grouped changelog entry for the distillation feature.
Checklist
Dependencies
Depends on
Earlier prerequisites
Parallel FastGen work
Conditional follow-up
If FastGen maintainers decline FastGen #44, we would close PhysicsNeMo #2100.
The preceding PhysicsNeMo stack would remain available.
Review process
Reviewers can compare this PR with its immediate parent. After a parent merges,
we update the remaining branches and their PR bases to keep each diff focused.