Run TimeMemorySummary.py in PR tests about cms-bot HOT 9 CLOSED

A new Issue was created by @fabiocos Fabio Cossutti.

@davidlange6, @Dr15Jones, @smuzaffar, @fabiocos, @kpedro88 can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

from cms-bot.

Maybe we could make a version of it that only prints the summary? The per-event info makes the logs huge and hard to read, and I usually don't find it very useful.

from cms-bot.

Ah, that is the summary version, distinct from TimeMemoryInfo.

from cms-bot.

We are not printing Time info per event, just memory jumps when they happen....
Still the trigger output summary is huge when written (and timing detail as well)

from cms-bot.

@smuzaffar in case, looking at the bot code, in particular https://github.com/cms-sw/cms-bot/blob/master/run-pr-tests#L640 this update would men adding to EXTRA_MATRIX_ARGS --command "--customise Validation/Performance/TimeMemorySummary.py" right?

from cms-bot.

@fabiocos , yes that is the correct place. As it is available in all release cycles so we can just hard code it here https://github.com/cms-sw/cms-bot/blob/master/run-pr-tests#L640

from cms-bot.

@smuzaffar should we consider also https://github.com/cms-sw/cms-bot/blob/master/pr_testing/test_multiple_prs.sh#L836 ?

from cms-bot.

yes that should be updated but I will ask @gudrutis to sync in other changes too. For now #1112 is good enough to go in.

from cms-bot.

I have merged #1112 and @gudrutis is going to fix the test_multiple_prs to include the changes.

from cms-bot.