Skip to content

[DOC] Add KNITRO #72

Open
eminyouskn wants to merge 5 commits intometab0t:masterfrom
eminyouskn:knitro-readme-clean
Open

[DOC] Add KNITRO #72
eminyouskn wants to merge 5 commits intometab0t:masterfrom
eminyouskn:knitro-readme-clean

Conversation

@eminyouskn
Copy link
Contributor

No description provided.

@metab0t
Copy link
Owner

metab0t commented Feb 11, 2026

Maybe more documentation should be added, like instructing the users to setup the solver and some solver-specific parameters?

A good example is the contribution of Xpress support: #62

@eminyouskn
Copy link
Contributor Author

I will work on that. Thanks @metab0t.

@eminyouskn
Copy link
Contributor Author

I checked and add KNITRO where I think is missing. Please let me know why if anything is missing.

@metab0t
Copy link
Owner

metab0t commented Feb 12, 2026

@eminyouskn
Copy link
Contributor Author

Thank you for pointing this out.
Regarding solver-specific attributes and parameters, I included everything I believe is relevant in the initial PR. Unfortunately, KNITRO callbacks are quite complex to handle properly, so I decided to leave them out of this first version of the implementation. I do plan to add support for them in the future; for that reason, I have not mentioned them in the documentation yet.
As for the license context manager, I am currently working on it and will add support soon. Once it is implemented, I will update the documentation accordingly.

@eminyouskn
Copy link
Contributor Author

@metab0t I just updated the doc of KNITRO to include the Env (license manager) documentation. I also mentionned that currently the callbacks are not supported. Please let me know if anything else is needed.

@eminyouskn eminyouskn changed the title Add KNITRO to README [DOC] Add KNITRO Feb 16, 2026
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

Successfully merging this pull request may close these issues.

2 participants