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

Wrong Riesz representation in the IPOPTSolver in Firedrake #188

Open
diego-hayashi opened this issue Dec 13, 2024 · 2 comments
Open

Wrong Riesz representation in the IPOPTSolver in Firedrake #188

diego-hayashi opened this issue Dec 13, 2024 · 2 comments

Comments

@diego-hayashi
Copy link

Due the commit firedrakeproject/firedrake@af2c9cc (Firedrake), the default Riesz representation changed from l2 to L2. This breaks the IPOPTSolver in Firedrake, since IPOPT is not prepared for anything other than l2.

One possible solution to this would be to specify the Riesz representation directly in the IPOPTSolver, as:

dJ = partial(self.rfn.derivative, forget=False, options={"riesz_representation": 'l2'})
@diego-hayashi diego-hayashi changed the title Wrong Riesz representation in the IPOPTSolver Wrong Riesz representation in the IPOPTSolver in Firedrake Dec 13, 2024
@dham
Copy link
Member

dham commented Dec 16, 2024

When you say "breaks" IPopt, what do you mean?

@diego-hayashi
Copy link
Author

IPOPT is still not ready for L2, and previous results are not reproducible the same way as before.

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