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

t_Bonmin_std.py fails on arm64, ppc64, ppc64el #2689

Open
pgrt opened this issue Jun 12, 2024 · 2 comments
Open

t_Bonmin_std.py fails on arm64, ppc64, ppc64el #2689

pgrt opened this issue Jun 12, 2024 · 2 comments
Assignees
Labels
Milestone

Comments

@pgrt
Copy link
Contributor

pgrt commented Jun 12, 2024

What happened?

I built openturns 1.23 on a Debian autobuilder equipped with debian/experimental, and on arm64, ppc64, ppc64el, the test t_Bonmin_std.py fails, see
https://buildd.debian.org/status/fetch.php?pkg=openturns&arch=arm64&ver=1.23-1&stamp=1718137756&raw=0

We get an error with "LIMIT_EXCEEDED" with the B-BB algorithm. Surprisingly:

  • this issue does not show up with the C++ test ;
  • with the other algorithms the test passes (e.g. by setting line 61 of t_Bonmin_std.py to for algo in ot.Bonmin.GetAlgorithmNames()[1:]:.

The issue does not show up with amd64, armhf, ...

How to reproduce the issue?

Build OT on a machine with one of the above architectures and debian/experimental.

Version

1.23

Operating System

Linux

Installation media

from source

Additional Context

No response

@pgrt pgrt added the bug label Jun 12, 2024
@jschueller
Copy link
Member

jschueller commented Jun 13, 2024

one difference I can see with the c++ test is the iteration limit, maybe try that:

diff --git a/python/test/t_Bonmin_std.py b/python/test/t_Bonmin_std.py
index 01ee596dd..82cb91039 100755
--- a/python/test/t_Bonmin_std.py
+++ b/python/test/t_Bonmin_std.py
@@ -55,6 +55,7 @@ problem.setInequalityConstraint(h)
 bonminAlgorithm = ot.Bonmin(problem, "B-BB")
 bonminAlgorithm.setStartingPoint([0, 0, 0, 0])
 bonminAlgorithm.setMaximumCallsNumber(10000)
+bonminAlgorithm.setMaximumIterationNumber(1000)
 bonminAlgorithm.setProgressCallback(progress)
 bonminAlgorithm.setStopCallback(stop)

@pgrt
Copy link
Contributor Author

pgrt commented Jun 13, 2024

Hi Julien,

Thanks, good catch! By removing this line, the C++ test fails.
By adding it as you suggest, the Python test passes.

jschueller added a commit to jschueller/openturns that referenced this issue Jun 13, 2024
@jschueller jschueller added this to the 1.24 milestone Jun 13, 2024
@jschueller jschueller self-assigned this Jun 13, 2024
@jschueller jschueller mentioned this issue Jun 13, 2024
jschueller added a commit to jschueller/openturns that referenced this issue Jun 17, 2024
jschueller added a commit to jschueller/openturns that referenced this issue Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants