Templar icon indicating copy to clipboard operation
Templar copied to clipboard

Global Statistics

Open wichtounet opened this issue 9 years ago • 4 comments

Hi,

As discussed in issue #6, I think it would be great to get some total statistics.

For starter, simply the total number of instantiations. Ideally, this number would be updated by the filters. This could also probably be done for each node type (template instantiation, memoization, deduced...).

The total duration could also be interesting. This would be the sum of the durations of all templates of first level in the graph.

Another thing that would be great but maybe harder to get is the number of instantiations of each template. For instance, the number of times std::forward has been instantiated.

What do you think ?

wichtounet avatar Jan 25 '16 10:01 wichtounet

All of these are reasonable. I cannot say how long it will take to implement each of them.

  • [x] total number of instantiations
  • [ ] -"- updated by filters
  • [x] -"- per node type (template instantiation, memoization, deduced...)
  • [x] total duration

schulmar avatar Jan 25 '16 12:01 schulmar

I added some of the statistics in the status bar, the tooltip gives more information.

Please comment on their usability.

schulmar avatar Feb 01 '16 12:02 schulmar

Hi,

I just tested the new branch. I really like it :) I think that these are very good metrics for comparison between different builds/versions.

Maybe the tooltip on duration is a bit too detailed or should be wrapped. Aside from this point, I think it is pretty good.

Using the status is imho the best way to achieve this. It is very visible and accessible.

Thanks a lot

wichtounet avatar Feb 02 '16 07:02 wichtounet

@wichtounet I removed the request for the number of instantiations per template. This is a larger point and should be in its own issue. Maybe this should even be in its own tool but I am not sure about that.

If you think that this functionality should be part of Templar, you can open a separate issue for it.

schulmar avatar Mar 17 '16 15:03 schulmar