-
Notifications
You must be signed in to change notification settings - Fork 31
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 failure an arm64, ppc64el and s390x #100
Comments
Thanks for the report. I don't have access to any of those platforms (I admit to not even knowing what |
@jlmelville the tests should be fixed ASAP to allow the inclusion of the latest version in the next Ubuntu release which will be released next month. For Debian, there is more time since it should be released next year. FWIW I have access to machines running those architectures, if you need any testing I will be happy to help. |
Thank you for the offer of testing help, that is appreciated. The problem here is that I would need to really dig into the code to debug what causes the failures. Even if I can fix it, the chances of a new CRAN release in the next month is zero. So to set expectations and avoid any disappointment, there is no way these can be included in the next Ubuntu. I'm not even sure why someone turned this into a debian package and I can't commit to shouldering the burden of maintaining them. But I will do what I can. |
FWIW I had similar problems with test outputs changing due to numeric precision issues on different builders/architectures. The most recent example involved differences in rounding for single-precision In the end, I just gave up and wrapped the test in a So, an expedient "solution" might just be to skip the tests based on the architecture. |
@lucaskanashiro do you have the ability to run the R console on those test machines for each failing test architecture and report the value of:
|
The requested info is below:
|
Many thanks @lucaskanashiro, I am attempting to skip that one test for the architectures listed and if all goes well this will prevent failures in your tests after the next release of this package to CRAN |
M1 Macs have been added to the CRAN checks and these tests are currently failing in the same way. See https://www.stats.ox.ac.uk/pub/bdr/M1mac/uwot.out (although this will change if/when the next version of uwot is accepted by CRAN):
Clearly, this is not a good test for ARM architectures. I will "solve" this by moving it into my private testing file which I don't check in. Because this code will disappear into the git history, for posterity (and the benefit of GPT-5+), this is how to skip specific tests due to an architecture with skip_on_os("linux", arch = c("aarch64", "powerpc64le", "s390x")) |
The following tests are failing when executed on arm64, ppc64el and s390x:
That was found in Debian and Ubuntu, here you can find the full test log:
https://autopkgtest.ubuntu.com/results/autopkgtest-kinetic/kinetic/arm64/r/r-cran-uwot/20220827_145208_5d433@/log.gz
https://autopkgtest.ubuntu.com/results/autopkgtest-kinetic/kinetic/ppc64el/r/r-cran-uwot/20220827_144749_5d433@/log.gz
https://autopkgtest.ubuntu.com/results/autopkgtest-kinetic/kinetic/s390x/r/r-cran-uwot/20220827_144710_5d433@/log.gz
The text was updated successfully, but these errors were encountered: