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

"Failure (test-leidenbase.R:253:5): modularity and significance return values" for architectures arm64, i386, ppc64el and s390x #36

Open
tillea opened this issue Oct 12, 2022 · 3 comments

Comments

@tillea
Copy link

tillea commented Oct 12, 2022

Hi,
the Debian packaged version of leidenbase has to pass its test on different architectures. This works well for amd64 and others. However, for some architectures there is a failure. You might like to look at the full log of the CI test for arm64. It contains this snippet which seems to be relevant:

BEGIN TEST testthat.R

R version 4.2.1 (2022-06-23) -- "Funny-Looking Kid"
Copyright (C) 2022 The R Foundation for Statistical Computing
Platform: aarch64-unknown-linux-gnu (64-bit)

R is free software and comes with ABSOLUTELY NO WARRANTY.
You are welcome to redistribute it under certain conditions.
Type 'license()' or 'licence()' for distribution details.

R is a collaborative project with many contributors.
Type 'contributors()' for more information and
'citation()' on how to cite R or R packages in publications.

Type 'demo()' for some demos, 'help()' for on-line help, or
'help.start()' for an HTML browser interface to help.
Type 'q()' to quit R.

> testthat::test_check("leidenbase")
Loading required package: leidenbase
[ FAIL 1 | WARN 0 | SKIP 0 | PASS 27 ]

══ Failed tests ════════════════════════════════════════════════════════════════
── Failure (test-leidenbase.R:253:5): modularity and significance return values ──
`t11_v01` not equal to `t11_v01_expect`.
1/1 mismatches
[1] 0.71 - 0.71 == 0.000133

[ FAIL 1 | WARN 0 | SKIP 0 | PASS 27 ]
Error: Test failures
Execution halted

This error occurs for the architectures arm64, i386, ppc64el and s390x.
For i386 there are even more errors ( here you can find the log ) but may be I should open a different issue once the issue above is fixed.
Kind regards, Andreas.

@tillea
Copy link
Author

tillea commented Jan 11, 2023

This issue for all other architectures than i386 was obviously solved in version 0.1.14. Unfortunately problems for i386 only are remaining which has the following failures:

══ Failed tests ════════════════════════════════════════════════════════════════
── Failure ('test-leidenbase.R:72'): ModularityVertexPartition membership and quality ──
isTRUE(all.equal(t02_v01, t02_v01_expect)) is not TRUE

`actual`:   FALSE
`expected`: TRUE
── Failure ('test-leidenbase.R:90'): RBERVertexPartition membership and quality ──
isTRUE(all.equal(t03_v01, t03_v01_expect)) is not TRUE

`actual`:   FALSE
`expected`: TRUE
── Failure ('test-leidenbase.R:126'): SurpriseVertexPartition membership and quality ──
isTRUE(all.equal(t05_v01, t05_v01_expect)) is not TRUE

`actual`:   FALSE
`expected`: TRUE
── Failure ('test-leidenbase.R:148'): resolution_parameter parameter ───────────
isTRUE(all.equal(t06_v01, t06_v01_expect)) is not TRUE

`actual`:   FALSE
`expected`: TRUE
── Failure ('test-leidenbase.R:166'): num_iter parameter ───────────────────────
isTRUE(all.equal(t07_v01, t07_v01_expect)) is not TRUE

`actual`:   FALSE
`expected`: TRUE
── Failure ('test-leidenbase.R:189'): initial_membership parameter ─────────────
isTRUE(all.equal(t08_v01, t08_v01_expect)) is not TRUE

`actual`:   FALSE
`expected`: TRUE
── Failure ('test-leidenbase.R:212'): edge_weights parameter ───────────────────
isTRUE(all.equal(t09_v01, t09_v01_expect)) is not TRUE

`actual`:   FALSE
`expected`: TRUE
── Failure ('test-leidenbase.R:235'): node_sizes parameter ─────────────────────
isTRUE(all.equal(t10_v01, t10_v01_expect)) is not TRUE

`actual`:   FALSE
`expected`: TRUE

[ FAIL 8 | WARN 0 | SKIP 0 | PASS 20 ]
Error: Test failures

I admit I have no idea how to cope with this.
Kind regards, Andreas.

@brgew
Copy link
Collaborator

brgew commented Jan 14, 2023 via email

@tillea
Copy link
Author

tillea commented Jan 14, 2023 via email

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

No branches or pull requests

2 participants