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

2.14.1: Tests fail: REQUIRE( coordinates(1) == Approx(0.0f).margin(0.01) ) #225

Closed
yurivict opened this issue Sep 3, 2020 · 4 comments
Closed

Comments

@yurivict
Copy link

yurivict commented Sep 3, 2020

]$ ./work/.build/ensmallen_tests
ensmallen version: 2.14.0 (No Direction Home)
armadillo version: 9.900.2 (Nocturnal Misbehaviour)

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
ensmallen_tests is a Catch v2.4.1 host application.
Run with -? for options

-------------------------------------------------------------------------------
SimpleAdaDeltaTestFunctionFMat
-------------------------------------------------------------------------------
/usr/ports/math/ensmallen/work/ensmallen-2.14.1/tests/ada_delta_test.cpp:66
...............................................................................

/usr/ports/math/ensmallen/work/ensmallen-2.14.1/tests/ada_delta_test.cpp:75: FAILED:
  REQUIRE( coordinates(1) == Approx(0.0f).margin(0.01) )
with expansion:
  -0.13114f == Approx( 0.0 )

OS: FreeBSD 12.1
clang-9

@UtR491
Copy link

UtR491 commented Sep 5, 2020

I think you should take a look at #182.

@zoq
Copy link
Member

zoq commented Sep 18, 2020

@yurivict any chance you can check if #182 works in some cases, if that's the case I'm going to bump the number of iterations.

@rcurtin
Copy link
Member

rcurtin commented Sep 20, 2020

@zoq I think bumping the iterations is the right thing to do here. Let me know what you think; if you want, I can put together a PR, up to you. 👍

@rcurtin
Copy link
Member

rcurtin commented Oct 5, 2020

Hey @yurivict, I believe this is fixed in #228. So hopefully in the next ensmallen release you will not see this problem. Thanks for the report! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants