mboehm7 commented on a change in pull request #1357: URL: https://github.com/apache/systemds/pull/1357#discussion_r682013116
########## File path: scripts/perftest/README.md ########## @@ -28,10 +28,16 @@ There are a few prerequisites: - Setup OpenBlas: <https://github.com/xianyi/OpenBLAS/wiki/Precompiled-installation-packages> - Install Perf stat: <https://linoxide.com/linux-how-to/install-perf-tool-centos-ubuntu/> -NOTE THE SCRIPT HAS TO BE RUN FROM THE ROOT OF THE REPOSITORY. +NOTE THE SCRIPT HAS TO BE RUN FROM THE PERFTEST FOLDER. +Examples: ```bash -./scripts/perftest/runAll.sh +./runAll.sh + +./runAll.sh perftestTemp MR Review comment: Sorry for the confusion - I think simply defining `CMD="./sparkDML.sh"` in `runAll.sh` would suffice - that way people can simply edit this once location if they want to run via systemds and everything becomes less convoluted (no more need to handle this in each script). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
