Ipopt 3.14 #6
Closed
Ipopt 3.14 #6
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- I guess the --vec_vals was meant for the following loop
- add some asserts to prevent null-pointer-dereference warnings - return on some functions early if working on empty vector/matrix - throw exception if ampl suffixes wrong - etc
- join makefiles and skip static non-distributed libraries - remove IPALLLIBS - remove duplicate am_conditional
- so show more officially that we assume that Fortran INTEGER is the same as C int
- some of them don't compile and I don't think they would support 64bit integers
- for hs071_f test and example
- pardiso-project's pardiso doesn't seem to have an option to switch to 64-bit integers nor does it look as if they had separate libs for that
- to avoid some confusion with smaller Archive link on left side, see coin-or-tools/ThirdParty-HSL#15
- ... when warmstart with same structure and grad_f_constant is set - need to get new gradient value still, as coefs may have changed for new solve
- requested in coin-or#611
- many thanks to @fschloesser on helping to figure these out
In the commented out code for writing the solution to a file, the third index is missing. Just adding it here.
- autoconf-archive 2021.02.19 -> 2022.09.03 - automake 1.16.3 -> 1.16.5 - libtool 2.4.6 -> 2.4.7
- for users that want to provide their own linear solver interface - fixes coin-or#637
- CSR_Format gives the upper triangular matrix, not the lower - see coin-or#638
- triplet gives lower diagonal, but CSR gives upper diagonal - see also coin-or#638
- These headers are required to use custom linear solvers on the user side. - Fixes coin-or#618, coin-or#637, coin-or#641.
|
@Robbybp let me know if you need more permissions to create a branch or merge this. |
|
Something is wrong with this branch. The IDAES-Ipopt copyright header is not present in the I will close this PR in favor of working out exactly what needs to be done on the issue IDAES/idaes-pse#1248, then will request the required permissions to do this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I just merged coin-or:stable/3.14 into our stable/idaes-3.13 branch. I'd like to create this as a new branch on this fork, but it seems I can't do that with a PR.