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

Inverse LSR1Operator #367

Open
MaxenceGollier opened this issue Mar 11, 2025 · 2 comments
Open

Inverse LSR1Operator #367

MaxenceGollier opened this issue Mar 11, 2025 · 2 comments

Comments

@MaxenceGollier
Copy link

We have an InverseLBFGSOperator but not an InverseLSR1 one, we should add it.

@dpo
Copy link
Member

dpo commented Mar 11, 2025

Since it’s likely to be indefinite, it will be of very limited use (if any). The forward operator is useful in trust-region methods. The inverse operator would be useful in a linesearch method, but since it won’t guarantee that we find a descent direction, I’m not sure what advantage it would have.

@MaxenceGollier
Copy link
Author

Ok, we should remove a few things in src/lsr1.jl and in the documentation as well then because I think it is confusing, I will open a PR.

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