search

gschlossnagle / yappi

Automatically exported from code.google.com/p/yappi
MIT License
0 stars 0 forks source link

bad performance of yappi because of overhead calculation #11

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
get_system_time() is executed twice pr function call to calculate yappi
overhead. This degrades the performance up to 2 or 3 times. Just remove
overhead calculation code.

Original issue reported on code.google.com by oktaka...@gmail.com on 7 Mar 2010 at 3:59

GoogleCodeExporter commented 8 years ago 
_call_leave calls tick() 2 times. Fix that. And also the real bottleneck is the
timing measumerement code for the profiler. As the number of functions being 
profiled
is increasing, more and more get_system_time() functions are called which 
degrades
performance. There is no easy way to optimize this, however, we can add some 
sort of
sampling value which can be passed to start() (note that this param should be 
an odd
number and less than the number of functions being profiled, otherwise 
profiling will
give crap information). This way, we can scale the runtime of the profiler for 
our
needs, tradeoff between accuracy/performance.

Original comment by oktaka...@gmail.com on 7 Mar 2010 at 7:01

GoogleCodeExporter commented 8 years ago 
updated in v0.43. See the v0.43 changes

Original comment by sum...@gmail.com on 10 Mar 2010 at 11:12

GoogleCodeExporter commented 8 years ago

Original comment by sum...@gmail.com on 12 Mar 2010 at 10:12