-
Notifications
You must be signed in to change notification settings - Fork 92
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
L2Reg error or misleading docs #94
Labels
Comments
|
Hello! Thanks for the report. Are you open to making a pull request for your requested document changes? In particular, if you're able to change the documentation for all the "basic proxs" listed in https://sigpy.readthedocs.io/en/latest/core_prox.html to be of the format "prox_{alpha g)(y) = argmin...", that would be much appreciated. |
I created a pull request |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In https://sigpy.readthedocs.io/en/latest/generated/sigpy.prox.L2Reg.html#sigpy.prox.L2Reg
is the proximal operator a function of y or z. It appears that the base class
sigpy.prox.Prox
uses y as the input to the prox operator. But y is a parameter for the L2Reg class called "bias".I think "z" is the bias based on what the description is. My tests confirm this.
See below for a test which passes based on the assumption that z is the bias parameter and y is the input to the prox operator.
Also the statement in the docs above says
min
instead ofargmin
.The text was updated successfully, but these errors were encountered: