Skip to content
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

Feature: Ipopt callbacks #157

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

jack-champagne
Copy link

Referenced the following during implementation:
https://github.com/jump-dev/Ipopt.jl/tree/4c156461ef1fda3c9f015520197afda4e8ca3e26?tab=readme-ov-file#solver-specific-callbacks
https://coin-or.github.io/Ipopt/IpStdCInterface_8h.html#ab86b55545fa3e0232a0e646409e0e82b

Add basic functionality, need to add:

  • test
  • docs
  • example

using internal callbacks, here are the primal variables as Ipopt optimizes. This was generated via saving the plots on the "Callback can get internal history" test

animation

Add basic functionality, need to add test, docs, and example
@@ -9,6 +9,7 @@ using ..Options

using NamedTrajectories
using MathOptInterface
using Ipopt
const MOI = MathOptInterface

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would be nice to add docstring for solve :)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely, just noting it here - adding some features to save best trajectory (on various parameters), access to variables saved inside of callbacks programmatically (get_callbacks)

@aarontrowbridge
Copy link
Member

could we add this gif to the docs somewhere

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