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.

Sign up for GitHub

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

Jump to bottom

More options in ipopt solverOptions #706

Open
EveCharbie opened this issue Jun 30, 2023 · 4 comments
Open

More options in ipopt solverOptions #706

EveCharbie opened this issue Jun 30, 2023 · 4 comments
Labels
good first issue Good for newcomers

Comments

@EveCharbie
Copy link
Collaborator

All the options should be implemented + a lot of the ones implemented are wrong. Example:

        @property
        def constr_viol_tol(self):
            return self._tol

Should be

        @property
        def constr_viol_tol(self):
            return self._constr_viol_tol

The complete list is here https://casadi.sourceforge.net/v1.9.0/api/html/dd/df1/classCasADi_1_1IpoptSolver.html
@EveCharbie EveCharbie added bug Something isn't working good first issue Good for newcomers labels Jun 30, 2023
@Ipuch Ipuch mentioned this issue Jul 6, 2023
Merged
@Ipuch Ipuch removed the bug Something isn't working label Jul 6, 2023
@Ipuch
Copy link
Collaborator

Ipuch commented Jul 6, 2023

At least I repaired the bug. Still relevant if we want more options.

@Ipuch Ipuch changed the title Error in ipopt options More options in ipopt solverOptions Jul 6, 2023
Ipuch added a commit to Ipuch/bioptim that referenced this issue Jul 19, 2023
@Ipuch
tracking tests on Ipopt options following pyomeca#715
ddfdaa1 
to have no more surprises pyomeca#706
@Ipuch Ipuch mentioned this issue Jul 19, 2023
Merged
@pariterre
Copy link
Member

@Alpha2Shahiri

@Alpha2Shahiri
Copy link
Contributor

It's already done, including the name. If you're referring to other ID descriptions, could you please specify which ones you're talking about?

@EveCharbie
Copy link
Collaborator Author

They are on the link provided.

Alpha2Shahiri added a commit to Alpha2Shahiri/bioptim that referenced this issue Nov 10, 2023
@Alpha2Shahiri
fix: pyomeca#706 Issue
842dc41 
More options in ipopt solverOptions (Renaming based on CasADi::IpoptSolver Class Reference)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pariterre @Ipuch @EveCharbie @Alpha2Shahiri