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

test/test_dmatrix - Increase tolerance and make test more robust #16

Closed
wants to merge 1 commit into from

Conversation

tamiko
Copy link
Contributor

@tamiko tamiko commented Jul 10, 2019

It turns out that on non-amd64 architectures the test test_dmatrix fails
due to differences in the compared matrices larger than floating point
roundoff errors [1]. Changing the limit from DBL_MIN to 1.0e4 *
DBL_EPSILON [2] (assuming that differences are indeed of the order of
eps) fixes the problem .

[1] https://launchpad.net/~ginggs/+archive/ubuntu/testing/+sourcepub/10330424/+listing-archive-extra
[2] https://en.cppreference.com/w/c/types/limits

It turns out that on non-amd64 architectures the test test_dmatrix fails
due to differences in the compared matrices larger than floating point
roundoff errors [1]. Changing the limit from DBL_MIN to 1.0e4 *
DBL_EPSILON [2] (assuming that differences are indeed of the order of
eps) fixes the problem .

[1] https://launchpad.net/~ginggs/+archive/ubuntu/testing/+sourcepub/10330424/+listing-archive-extra
[2] https://en.cppreference.com/w/c/types/limits
@cburstedde
Copy link
Owner

Thanks! Can you please commit a file doc/author_maier.txt to put your changes to libsc either into the public domain or under the FREEBSD license?

@cburstedde
Copy link
Owner

Hi, bumping this for the requested author text file. Thanks!

@holke
Copy link
Contributor

holke commented Mar 25, 2020

Hi.
I can confirm that the dmatrix test fails on an Intel i7-6600U when configured with the C flag

-march=native

Strangely though, it only occurs when running

make -j check

and not with

make check

(any idea why?)

This fix resolves the error.

@cburstedde
Copy link
Owner

The dependency on -j is weird, but addressed by the fix.

I'll merge it but I cannot do that until I the author_tamiko.txt file is added to put the code under the FreeBSD license.

@cburstedde
Copy link
Owner

I'll merge it but I cannot do that until I the author_tamiko.txt file is added to put the code under the FreeBSD license.

Hi can I ping you again about this little file? Thanks!

@cburstedde
Copy link
Owner

Thanks for your proposal! I'm closing this PR though since I don't have the author file.

@cburstedde cburstedde closed this Feb 23, 2021
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.

None yet

3 participants