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

Flaky failure of TestSLSQP::test_minimize_unbounded_approximated on x86_64 #238

Closed
musicinmybrain opened this issue Nov 4, 2023 · 2 comments

Comments

@musicinmybrain
Copy link
Contributor

While working on updating the python-cyipopt package in Fedora Linux to 1.3.0, I observed that on x86_64 (only—not on aarch64, ppc64le, or s390x—although with i686 not tested), TestSLSQP::test_minimize_unbounded_approximated frequently fails. I observed that two out of five scratch-builds failed.

=================================== FAILURES ===================================
________________ TestSLSQP.test_minimize_unbounded_approximated ________________
self = <cyipopt.tests.unit.test_scipy_ipopt_from_scipy.TestSLSQP object at 0x7fc6d5629c70>
    def test_minimize_unbounded_approximated(self):
        # Minimize, method=None: unbounded, approximated jacobian.
        jacs = [None, False]
        for jac in jacs:
            res = minimize(self.fun, [-1.0, 1.0], args=(-1.0, ),
                           jac=jac, method=None,
                           options=self.opts)
>           assert_(res['success'], res['message'])
E           AssertionError: b'Algorithm stopped at a point that was converged, not to "desired" tolerances, but to "acceptable" tolerances (see the acceptable-... options).'
cyipopt/tests/unit/test_scipy_ipopt_from_scipy.py:158: AssertionError

The scipy package is at version 1.11.1. I’m happy to run any tests or add any other details that might be helpful.

@moorepants
Copy link
Collaborator

moorepants commented Nov 26, 2023

Fixed in a2d7dd5

@moorepants
Copy link
Collaborator

I decided to just allow the "failure", as it does converge to the acceptable_tol, just not the desired tolerance.

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