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

Casadi and ipopt #57

Open
jcarpent opened this issue Aug 22, 2021 · 4 comments
Open

Casadi and ipopt #57

jcarpent opened this issue Aug 22, 2021 · 4 comments

Comments

@jcarpent
Copy link

Currently, casadi is provided with Ipopt support.
Would it be possible to make this external to casadi. In other words, having casadi and casadi-ipopt outputs of this package?

@jcarpent jcarpent mentioned this issue Aug 22, 2021
5 tasks
@traversaro
Copy link
Contributor

Would it be possible to make this external to casadi. In other words, having casadi and casadi-ipopt outputs of this package?

That could be a good idea, but I would suggest to have a casadi-core or similar for casadi without ipopt (or any other solver), casadi-ipopt for the casadi_nlpsol_ipopt library, and casadi package that depends on both packages. In this way, we could permit to depend on casadi-core, while keeping the semantics of the existing casadi package, that installs ipopt as well.

@traversaro
Copy link
Contributor

However, if the goal is just to merge #56, an easier alternative may be to just skip the ipopt dependency on arm-osx, and then add it back once ipopt is available for arm-osx .

@jcarpent
Copy link
Author

Would it be possible to make this external to casadi. In other words, having casadi and casadi-ipopt outputs of this package?

That could be a good idea, but I would suggest to have a casadi-core or similar for casadi without ipopt (or any other solver), casadi-ipopt for the casadi_nlpsol_ipopt library, and casadi package that depends on both packages. In this way, we could permit to depend on casadi-core, while keeping the semantics of the existing casadi package, that installs ipopt as well.

Seems to be nice and compatible with existing libs.

@jcarpent
Copy link
Author

However, if the goal is just to merge #56, an easier alternative may be to just skip the ipopt dependency on arm-osx, and then add it back once ipopt is available for arm-osx .

Yes it will help #56. But at the same time, there is a lot of potential packages (e.g pinocchio-casadi) which needs casadi but does not depend of casadi-ipopt, which would come with further and useless downloads or dependencies.

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

No branches or pull requests

2 participants