-
Notifications
You must be signed in to change notification settings - Fork 407
Improve lead time support for diffusion models #980
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
Merged
Merged
Changes from 16 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
bbaea98
Improve lead time support for diffusion models
jleinonen 232c29c
Update changelog
jleinonen 8b94e59
Add back mistakenly removed docstrings and type hints
jleinonen f3338b6
Revert a couple more unintended changes
jleinonen 7ba0563
Fix type hint of lead time label
jleinonen 11e4ea5
Fix deterministic samples to allow CorrDiff tests to pass
jleinonen 6f08b02
Rename utils.generative to utils.diffusion
jleinonen 63b70c9
Merge main and resolve conflicts
jleinonen fd35097
Add back __init__.py in generative
jleinonen ef8a9a6
Revert unnecessary changes
jleinonen d0b1bfb
Revert unnecessary changes
jleinonen 4d503c4
Revert unnecessary changes
jleinonen 77a5cde
Merge branch 'NVIDIA:main' into leadtime-fixes
jleinonen e600803
Merge branch 'NVIDIA:main' into leadtime-fixes
jleinonen 4929596
Merge branch 'main' into leadtime-fixes
CharlelieLrt 457d50d
Minor docstring improvement in SongUNetPosEmdb
CharlelieLrt 522440b
Add value checks and docstrings
jleinonen fd96890
Update docstrings, add error condition
jleinonen 29ca853
Update docstrings
jleinonen e87c4c5
Fix lead time tests
jleinonen e2fdd39
Fix tests after merge
jleinonen 6cec691
Merge branch 'NVIDIA:main' into leadtime-fixes
jleinonen bf461a8
Update docstring
jleinonen 89a5bd1
Change super().__init__ to use keyword args
jleinonen 2c0cd29
Minor formatting in deterministic_sampler docstring
CharlelieLrt 77b162b
Minor renaming and formatting in loss.py
CharlelieLrt 761468f
Removed dtype casting of pos_emb in SongUNetPosEmbd
CharlelieLrt 948063e
Removed duplicate code in SongUNetPosEmbd.positional_embedding_indexing
CharlelieLrt 58c55af
Refactor positional_embedding_indexing to eliminate dead and duplicat…
CharlelieLrt fe3052b
Refactor positional_embedding_selector to enable batched lead-time la…
CharlelieLrt b5992d5
Moved new test from song_unet_pos_embd to song_unet_pos_lt_embd
CharlelieLrt ebaa43f
Updated CHANGELOG.md
CharlelieLrt 2013cf5
Added safety check to force users to use SongUNetPosLtEmdb for lead-t…
CharlelieLrt 374c9f8
Deleted unecessary test
CharlelieLrt 2b6ac4e
Fixed bug in positional_embedding_selector + changed samplers and tes…
CharlelieLrt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.