-
Notifications
You must be signed in to change notification settings - Fork 32
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
Diagonal quasi-Newton operators #237
Conversation
Codecov ReportBase: 97.61% // Head: 97.38% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #237 +/- ##
==========================================
- Coverage 97.61% 97.38% -0.24%
==========================================
Files 13 14 +1
Lines 965 994 +29
==========================================
+ Hits 942 968 +26
- Misses 23 26 +3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
A few more...
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.
Some more comments that should simplify things.
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.
Last ones.
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.
Actually, test_diag
should go under test
. Include it in runtests.jl
.
a5530b1
to
a1fabed
Compare
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.
Hi, thank you for the PR. A few comments:
@geoffroyleconte Should be all fixed. Do you mind taking another quick look? |
It's expected that FreeBSD/nightly fail at the moment. |
Thank you @Joshua-Wolff and @geoffroyleconte ! |
Introduce new Abstract type in lbfgs.jl