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

Convert SPDHG to CIL optimization #1

Open
mehrhardt opened this issue Jul 26, 2018 · 8 comments
Open

Convert SPDHG to CIL optimization #1

mehrhardt opened this issue Jul 26, 2018 · 8 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@mehrhardt
Copy link


use A(x) <=> A.direct(x)
use A.adjoint(x, out=y) <=> y = A.adjoint(x)
use f.proximal(tau)(x) <=> f.prox(x, tau)
f.convex_conj <=> f_cvx

@mehrhardt mehrhardt self-assigned this Jul 26, 2018
@mehrhardt
Copy link
Author

This has been implemented by 53453fe. The code is not really documented nor well written. Moreover, there are several examples missing. Finally, the more code is needed to add regularization to it, for example from CCPi.

@mehrhardt mehrhardt added the help wanted Extra attention is needed label Jul 27, 2018
@paskino
Copy link
Contributor

paskino commented Jul 29, 2018

I guess we could add proximal rather than prox and it's a good idea to add out=y.

@paskino
Copy link
Contributor

paskino commented Jul 29, 2018

I added the CIL optimisation and regularisation to SIRF SuperBuild SyneRBI/Hackathon-SIRF-SuperBuild#1

@mehrhardt
Copy link
Author

I guess we could add proximal rather than prox and it's a good idea to add out=y.

@paskino, yes, that would be a good idea! This way CIL is also more compatible to ODL which might turn out to be useful at some point.

@paskino
Copy link
Contributor

paskino commented Jul 30, 2018

@KrisThielemans
Copy link
Member

@mehrhardt could you create an issue in SIRF (not hackathon) with where the out syntax is needed? It could be initially done as a work-around (for compatibility), but for PET it should be easy.

Please also say what out is supposed to do. should it check if sizes etc are as expected? What to do if they're not? Should it overwrite or add to existing data?

Possibly point us to the ODL doc, or we can get some ODL people involved.

@mehrhardt
Copy link
Author

See SyneRBI/SIRF#194

@mehrhardt
Copy link
Author

mehrhardt commented Oct 18, 2018

This issue can be closed as 5efd25a completes its implementation and simple example showing its usage. However, all code for SPDHG is far from tidied up and should be included at appropriate places in SIRF and/or CIL. I created an issue for this vais-ral/CCPi-Framework#154.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants