-
Notifications
You must be signed in to change notification settings - Fork 21
Add support for KNITRO #71
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
Conversation
|
The implementation is high quality, especially your design of An annoying issue is that when using the license you sent to me, knitro outputs during |
|
Thanks. You can remove this by setting the variable |
|
@metab0t I finished my cleaning. |
|
LGTM, I will merge it to see whether the CI is happy. |
|
@eminyouskn Please fix the compilation errors on Linux and mac according to the latest CI logs. It seems to be related to constexpr initialization and some type conversions. |
|
@metab0t thanks for merging. Taking a look at it now. |
|
@eminyouskn It turned out to be small issues where constexpr string is not supported by old compilers. They are fixed now. I notice that some documentation should also be modified to reflect the added support for KNITRO, and you can add them in subsequent PRs. |
|
Nice thank you again! |
No description provided.