-
Notifications
You must be signed in to change notification settings - Fork 4
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
Allow to use other preconditioners than ICFS #12
Conversation
@@ -4,7 +4,7 @@ mutable struct ExaTronProblem{VI, VD} | |||
nnz_a::Integer # number of Hessian entries in the strict lower | |||
A::AbstractTronMatrix | |||
B::AbstractTronMatrix | |||
L::AbstractTronMatrix | |||
L::AbstractPreconditionner | |||
indfree::VI # a working array of dimension n |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might want to change variable name, since it is not just a lower triangular matrix now and has L inside.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That makes sense. Would you prefer P
, or something more explicit, as precond
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not mind what it is called, as long as it is not L.
1110c05
to
1fed1a7
Compare
1fed1a7
to
bc150f8
Compare
@youngdae do you think we could merge this PR into master? |
Sorry, I was a bit behind. I'll merge it soon. |
Thanks ! |
No description provided.