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

Rscript throws error "missing value where TRUE/FALSE needed" when analyzing benchmarks #31045

Open
trivikr opened this issue Dec 20, 2019 · 2 comments
Labels
benchmark Issues and PRs related to the benchmark subsystem.

Comments

@trivikr
Copy link
Member

trivikr commented Dec 20, 2019

  • Version: master
  • Platform: Ubuntu
  • Subsystem: http2

Followed documentation to create comparison benchmarks

Command run:

$ node benchmark/compare.js --old ../node-master --new ../node-http2-for-of --set benchmarker=wrk http2 > compare-http2.csv
[05:52:54|% 100| 5/5 files | 60/60 runs | 12/12 configs]: Done

Output file (rename extension to CSV): compare-http2.txt

I receive the following error when trying to analyze benchmarks:

$ cat compare-http2.csv | Rscript benchmark/compare.R
Error in if (w$p.value < 0.001) { : missing value where TRUE/FALSE needed
Calls: ddply ... llply -> loop_apply -> .Call -> <Anonymous> -> .fun
Execution halted
@trivikr
Copy link
Member Author

trivikr commented Dec 20, 2019

The line where error is thrown

node/benchmark/compare.R

Lines 81 to 83 in 8b4af64

if (w$p.value < 0.001) {
confidence = '***';
} else if (w$p.value < 0.01) {

@trivikr
Copy link
Member Author

trivikr commented Dec 20, 2019

This is probably related to using benchmarker=wrk in the command, and can be closed.

I would still prefer to keep it open to implement benchmark exit if data is not useful.

@targos targos added the benchmark Issues and PRs related to the benchmark subsystem. label Dec 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
benchmark Issues and PRs related to the benchmark subsystem.
Projects
None yet
Development

No branches or pull requests

2 participants