search

mirage / bechamel

Agnostic benchmark in OCaml (proof-of-concept)
MIT License
44 stars 15 forks source link

Typo in `benchmark.ml` ? #11

Closed lyrm closed 3 years ago

lyrm commented 3 years ago

Hi, Should the value used as the maximum number of times the benchmarked function is run actually be !run instead of !idx ? Link here Copy of the line :

 while (not (exceeded_allowed_time cfg.quota init_time)) && !idx < cfg.run do
dinosaure commented 3 years ago

Yes, you are right, I will propose a fix.