-
Notifications
You must be signed in to change notification settings - Fork 92
autoregressive solver #731
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
base: 0.3
Are you sure you want to change the base?
autoregressive solver #731
Conversation
|
Hi @Davide-Miotti , thank you for your valuable contribution! I would suggest integrating the entire |
765550f to
1daa10b
Compare
|
To give some additional context, here is a python script that uses the code from this PR: |
|
With the last commit I deleted the autoregressive condition and now all the logic is contained within autoregressive solver, I also pushed by mistake a test script, called |
|
Hi @GiovanniCanali @Davide-Miotti, since this PR is related to 0.3 release, I changed the base branch |
1bb9d46 to
76e5925
Compare
Description
Implements
AutoregressiveCondition,TimeWeightingInterfaceandAutoregressiveSolver.This is a tentative initial implementation, not a definive development.
AutoregressiveConditioninput: Initial state (unroll: Ground truth sequence (time_weightingstrategy (e.g., linear decay) directly within the condition.AutoregressiveSolverInterfaceAutoregressiveSolverinput(unrolltarget (unroll_length.Limitations
This PR fixes #730
Checklist