-
Notifications
You must be signed in to change notification settings - Fork 15
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
Implement FAASTA #45
Comments
@zaccharieramzi are you planning a PR for this issue? |
Yes I am. I am currently working on the best way to package it but it's basically done. Do you want to have a WIP PR to see if the direction is statisfying? |
@zaccharieramzi I think having a diverse set of reconstruction algorithms would be useful regardless. So if you have already implemented it then feel free to open a PR. |
Ok I will package it in a PR. |
@zaccharieramzi are you still planning a PR for this algorithm? |
Not at the moment. Do you still want to keep this open? I will probably do it at some point but I don't think I can find the time right now. |
We can keep it open, I just wanted to check on the status. :) |
The FAASTA algorithm allows us to perform a FISTA algorithm in the analysis formulation. In our case we could see if the FAASTA still performs well with the new restarting options.
Since the FISTA with an orthogonal transform is way faster than the Condat-Vu, we are going to see now if this holds true even when we have to compute an extra inner iteration.
The text was updated successfully, but these errors were encountered: