danielpalme / IocPerformance

Performance comparison of .NET IoC containers
https://danielpalme.github.io/IocPerformance
Apache License 2.0
874 stars 157 forks source link

Remove warmup from benchmark #153

Closed Wsm2110 closed 1 year ago

Wsm2110 commented 1 year ago

Warmup will result in caching of delegates and compiling of all expressions. This will give containers with few features an unfair advantage over others.

I.e. warmup of 5 benchmarks will lead to for instance 20 cached entries. A warmup of every benchmark will lead to 70, resulting in more hashcollissions
This will cause benchmarks to interfere with others. Removing the warmup will cause benchmarks to be more accurate

danielpalme commented 1 year ago

I don't think it makes any sense to put further effort into this benchmark. It's outdated and has to be rewritten for .NET Core.

Wsm2110 commented 1 year ago

Well, i guess you should archive this repo. it's outdated and the benchmarks are not accurate