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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃殌[FEA]: CorrDiff training/inference without regression #478

Open
jleinonen opened this issue Apr 30, 2024 · 2 comments
Open

馃殌[FEA]: CorrDiff training/inference without regression #478

jleinonen opened this issue Apr 30, 2024 · 2 comments
Assignees
Labels
? - Needs Triage Need team to review and classify enhancement New feature or request

Comments

@jleinonen
Copy link
Collaborator

Is this a new feature, an improvement, or a change to existing functionality?

New Feature

How would you describe the priority of this feature request

Medium

Please provide a clear description of problem you would like to solve.

For comparison purposes it would be good to have the ability to train CorrDiff with a plain diffusion model without regression.

Describe any alternatives you have considered

It's possible to do this by plugging in a "regression" model that returns all zeros but this is a rather hacky solution.

@jleinonen jleinonen added enhancement New feature or request ? - Needs Triage Need team to review and classify labels Apr 30, 2024
@jleinonen jleinonen changed the title 馃殌[FEA]: Training/inference without regression 馃殌[FEA]: CorrDiff training/inference without regression Apr 30, 2024
@ram-cherukuri
Copy link
Collaborator

@jleinonen Can you provide more context - If we dont need a regression step, wouldn't this be like training a diffusion model which Modulus already supports a AI researcher from doing?

@jleinonen
Copy link
Collaborator Author

@jleinonen Can you provide more context - If we dont need a regression step, wouldn't this be like training a diffusion model which Modulus already supports a AI researcher from doing?

@ram-cherukuri We have an example for a diffusion model but it doesn't have nearly all the functionality that the CorrDiff code has now. Whereas disabling the regression step in the CorrDiff codebase should be a matter of a few simple changes according to @mnabian.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
? - Needs Triage Need team to review and classify enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants