Draft: Put tsub column at the end, to avoid breaking parsing scripts

As discussed with @furmento.

Note that it is not really possible to "put new columns at the end of each line", I just put the tsub column after the gflops one, I hope it suits your needs.

Honestly, I'm not for it. It does not make any sense when you read it. Here you interleave times and flops, and what about the check column that is added when you enable --check. This one is always shifted to the right.

i think it will be easier if i update my data. changing the column order may be a problem for other cases. i'll deal with it.

Here you interleave times and flops

I agree with you. Maybe a simpler solution is to display the tsub column only when --async or --splitsub options are provided ? (I don't know if it's possible to easily implement this solution)

I think that would be the best way, and that's what we wanted to do at the beginning, but I don't remember why we could not.

Do we cancel this PR, or do we change it to an issue to think about potentially enabling/disabling columns ?

i'd say to turn it into an issue

Yes!

created #106 to continue this discussion

See #106

resolved all threads

mentioned in issue #106

marked this merge request as draft

closed