plotman icon indicating copy to clipboard operation
plotman copied to clipboard

Include copy time on plotman analyze

Open ReaktorDTR opened this issue 4 years ago • 3 comments

Closes #673 Added: Copy time column in analyzer Updated: Total time includes Copy time

ReaktorDTR avatar May 30 '21 15:05 ReaktorDTR

I'm a bit hesitant to have total time in analyze mean a different thing than total time in the logs. Any ideas what else would be a good name for the combination? Having total, copy, and combined time would allow the output code to stay more like before but just add the two new bits.

Do you have a log you could share that shows the copy time? My most recent logs don't include it. They should have been from 1.1.7. I have tmp the same as dst so there's no copying so it's understandable.

altendky avatar Jun 20 '21 19:06 altendky

It might be called Summary. I don't have logs for 1.1.7. When I got the first numbers in the Copy Time column I made a decision to switch from separate folders to one for all 😄

ReaktorDTR avatar Jun 20 '21 19:06 ReaktorDTR

How about we just go literal with start + copy. I think that still ends up about the same length as the data in the table cells. If there's inspiration later it can change.

altendky avatar Jun 20 '21 23:06 altendky