-
Notifications
You must be signed in to change notification settings - Fork 202
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
collect avg timings for module subcmds #946
base: develop
Are you sure you want to change the base?
Conversation
better formatting for timings for module commands
Conflicts: easybuild/tools/modules.py
…ework into time_module_subcmd Conflicts: easybuild/tools/modules.py
…if $EASYBUILD_TEST_MODULE_CMD_TIMINGS is defined, use setdefault on MODULE_SUBCMD_TIMINGS
@rtmclay: I cleaned this up a bit further, to make sure the timings are only collecting when testing, and that they're only printed when requested (i.e. when @JensTimmerman, @stdweird: please review? |
Looks good |
Is this really something you want to just print out? Imho it's better to actually create a benchmark for that tool then, instead of pushing this in the master unittests. The speed of the module commands should not be something the unittests are concerned with. |
EasyBuild framework unit test suite FAILed. See https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2923/console for more details. Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do. |
Timings collected with full test suite with EasyBuild v2.8.0dev (very close to v2.7.0), cfr https://travis-ci.org/boegel/easybuild-framework/builds/119748444. main conclusion: waaaay too many
|
EasyBuild framework unit test suite FAILed. See https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2938/console for more details. Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do. |
@boegel Relevance of this one? |
Ping @boegel again |
This needs more work before it can go in, this is just a (useable) mockup.
@rtmclay is using this to work on speeding up
--terse avail
.