Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make plain U-net available on score_sde_pytorch side #35

Merged
merged 4 commits into from
Aug 9, 2024

Conversation

henryaddison
Copy link
Owner

Now that a way to train determinstic models using the same training loop as the score sde models, add a way to use the same u-net implementation as on the deterministic package (so eventually can remove that seperate deterministic package)

Plus add some helper scripts for testing on JASMIN

@henryaddison henryaddison force-pushed the u-net-score-module branch 8 times, most recently from cb37f44 to b23301f Compare August 9, 2024 11:00
in theory this should replace the mirroring deterministic package
but obviously we should test this once blue pebble is back up properly

also include configs for a now more tuned config for plain det unet
and one that resembles the old det unet config more closely
so can basically disable EMA with a flag.
This is another difference between u-net trained on score_sde side deterministically
and the separate deterministic training approach.""

In theory decay rate of 1 should allow this but it's complicated by a num_updates params too
a rate of 1 means no EMA
this is backwards compatible unlike adding a new config attribute
@henryaddison henryaddison merged commit 468fa7f into deterministic-ncsnpp Aug 9, 2024
3 checks passed
@henryaddison henryaddison deleted the u-net-score-module branch August 9, 2024 11:06
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.

1 participant