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

Add support for proximal operator proposed in "Convex Low Rank Approximation" by Larsson and Olsson #51

Open
marcusvaltonen opened this issue Mar 15, 2022 · 5 comments

Comments

@marcusvaltonen
Copy link
Contributor

In [1] a general low-rank inducing framework was introduced and showed many benefits compared to e.g. the nuclear norm when applied to common computer vision tasks. This was later generalized in [2].

Suggestion on enhancement

Implement [2] and treat the special cases R_g, R_mu and R_i=k, separately. (See papers for details).

References

[1] Larsson and Olsson, "Convex Low Rank Approximation", International Journal of Computer Vision (IJCV), 2016
[2] Valtonen Ornhag and Olsson, "A unified optimization framework for low-rank inducing penalties", In the Proceedings of the IEEE/CVF Conference on Computer Vision and Pattern Recognition (CVPR), 2020

@mrava87
Copy link
Contributor

mrava87 commented Mar 22, 2022

Same as this other issue, I will be happy for us to support this.

I actually have a project starting soon where we would like to look more into these algorithms in the context of geophysical inverse problems, so I will be very happy to team up to get these methods in pyproximal so we can easily compare them against each other in a easy way :)

@marcusvaltonen
Copy link
Contributor Author

I can look into this. I just want to clarify that these penalties work in both the linear and bilinear setting, so they are independent of second-order methods and can be implemented in no particular order.

@marcusvaltonen
Copy link
Contributor Author

The R_mu penalty is the f_mu (quadratic envelope of the cardinality function) acting on the singular values. There is a PR for this penalty open #71

@marcusvaltonen
Copy link
Contributor Author

I am still working on a tutorial for this. Unfortunately, I have to prioritize other things, but I expect to have something by the end of the summer.

@mrava87
Copy link
Contributor

mrava87 commented Jun 16, 2022

No worries, take your time :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants