sofa-framework / sofa

Real-time multi-physics simulation with an emphasis on medical simulation.
https://www.sofa-framework.org
GNU Lesser General Public License v2.1
867 stars 296 forks source link

[Simulation.Core] Remove useless and annoying timers #4631

Closed alxbilger closed 3 weeks ago

alxbilger commented 4 weeks ago

Useless:

For example, the timer in UpdateMappingVisitor::processMechanicalMapping does not measure anything.

Annoying:

I don't think it is a good practice to add timer on individual components. In particular, the ones I removed measure neglectable durations. The fact that there is one timer for every component adds a lot of pollution in the analysis of the timings, for nothing because it is neglectable.


By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).


Reviewers will merge this pull-request only if

alxbilger commented 3 weeks ago

[ci-build][with-all-tests]

alxbilger commented 3 weeks ago

@epernod Thanks for the review. I added a description on the PR