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

opt.reg is not used in phaselift #7

Open
ZongyuLi-umich opened this issue May 10, 2022 · 0 comments
Open

opt.reg is not used in phaselift #7

ZongyuLi-umich opened this issue May 10, 2022 · 0 comments

Comments

@ZongyuLi-umich
Copy link

In

muFinal = opts.regularizationPara; % Regularization Parameter

where we say

muFinal = opts.regularizationPara;   % Regularization Parameter

but later in the code we define

mu = norm(eig(AtL(b0)),1)*opts.tol;

And then we use mu everywhere,
so I don't see how muFinal is useful in this code.

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

1 participant