Skip to content

Weaken test to improve flakiness#213

Open
MichaelChirico wants to merge 3 commits into
astamm:masterfrom
MichaelChirico:patch-1
Open

Weaken test to improve flakiness#213
MichaelChirico wants to merge 3 commits into
astamm:masterfrom
MichaelChirico:patch-1

Conversation

@MichaelChirico

Copy link
Copy Markdown

Closes #212 by reducing flakiness to ~.1%

It seems the actual flakiness is higher, and maxeval is always reached (not xtol_rel).
@aadler

aadler commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Maybe drop the tolerance back to 2e-2 again, given that four platforms are failing?

@MichaelChirico

MichaelChirico commented Jul 1, 2026

Copy link
Copy Markdown
Author

Hmm those errors look unrelated; I reproduce them on master:

  File ‘nloptr/libs/nloptr.so’:
    Found ‘_ZSt4cerr’, possibly from ‘std::cerr’ (C++)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘_ZSt4cout’, possibly from ‘std::cout’ (C++)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘__printf_chk’, possibly from ‘printf’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘abort’, possibly from ‘abort’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘exit’, possibly from ‘exit’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘putchar’, possibly from ‘putchar’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘puts’, possibly from ‘printf’ (C), ‘puts’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘stderr’, possibly from ‘stderr’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
  
  Compiled code should not call entry points which might terminate R nor
  write to stdout/stderr instead of to the console, nor use Fortran I/O
  nor system RNGs nor [v]sprintf.
  
  See ‘Writing portable packages’ in the ‘Writing R Extensions’ manual.

@aadler

aadler commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Hmm those errors look unrelated, are they present on master?

  File ‘nloptr/libs/nloptr.so’:
    Found ‘_ZSt4cerr’, possibly from ‘std::cerr’ (C++)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘_ZSt4cout’, possibly from ‘std::cout’ (C++)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘__printf_chk’, possibly from ‘printf’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘abort’, possibly from ‘abort’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘exit’, possibly from ‘exit’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘putchar’, possibly from ‘putchar’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘puts’, possibly from ‘printf’ (C), ‘puts’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
    Found ‘stderr’, possibly from ‘stderr’ (C)
      Object: ‘nlopt/lib/libnlopt.a’
  
  Compiled code should not call entry points which might terminate R nor
  write to stdout/stderr instead of to the console, nor use Fortran I/O
  nor system RNGs nor [v]sprintf.
  
  See ‘Writing portable packages’ in the ‘Writing R Extensions’ manual.

I think those may be the issues with which @astamm is dealing with the nlopt sources themselves.

@astamm

astamm commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Yes. I have a PR under review here: stevengj/nlopt#608.
Once merged, I am planning to move towards a bundling strategy as in: https://github.com/astamm/nloptrbundled, which takes care of these warnings.

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

Successfully merging this pull request may close these issues.

NLOPT_GN_ESCH test is flaky (.6%)

3 participants