Skip to content
This repository was archived by the owner on Dec 6, 2023. It is now read-only.

Conversation

@fabianp
Copy link
Member

@fabianp fabianp commented Nov 5, 2016

Also includes an example of using this proximal operator for denoising. The output of this example is

image

Other changes:

  • Added tolerance parameter for FISTA

@mblondel
Copy link
Member

mblondel commented Nov 6, 2016

Nice!

I have recently worked on a more efficient implementation of FISTA:
https://gist.github.com/mblondel/5105786d740693a6996bcb8e482c7083

The main advantage is that it is possible to compute the projection and regularization term in the same function call, which allows to share some expensive computations in some cases (e.g. nuclear norm). I also support a tolerance constant.

@mblondel
Copy link
Member

mblondel commented Nov 7, 2016

@fabianp I am not very familiar with the TV proximity operator. Perhaps you could ask someone else who is to give a quick review?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants