arbtt-graph icon indicating copy to clipboard operation
arbtt-graph copied to clipboard

Published 20 hours ago • verified publisher icon rejuvyesh

Omit "total time" rows

Open jbriales opened this issue 4 years ago • 1 comments

Fixes #6

It seems in my arbtt-stats version (arbtt-stats 0.10.2), there are rows with 'total time' as inctag which currently readArbttStats.py does not expect.

I'm not sure if there is a better solution, but filtering from inside readArbttStats.py for these cases just works.

I'm not sure why the total time is being dumped to the CSV files, as we're not using the --total-time option in arbtt-stats. I tried running with this to see the difference but the binary then fails.

Another option might be to filter for those entries from within update, but I'm not sure about the right syntax in FILTER.

jbriales avatar Apr 13 '20 13:04 jbriales

Ah, from arbtt's changelog, Version 0.10.2:

The category report now also has an entry (total time), especially useful with --for-each.

Yeah, we aren't using this in arbtt-graph so should be alright to filter it out. Can you add a comment explaining this in the file? I'll merge it then.

rejuvyesh avatar Apr 13 '20 21:04 rejuvyesh