Re: ftbench update: make integrated

Ahmet Göksu <[email protected]>
Newsgroups gmane.comp.fonts.freetype.devel
Message-ID <cfa59a4a-e589-421b-9063-a5d2713dd3f2@Spark>
Hi,
I have done the changes you want.
> 36.5% run difference is baaaad. AFAICS, you haven't yet worked on
> omitting 'warmup' iterations, right?
I am planning to increase the iteration count by 10% and ignore the results for them. Trying to figure out the benchmarking program but actually drowning in 1500 lines of code.

Best,
Goksu
goksu.in
On 31 Jul 2023 07:53 +0300, Werner LEMBERG <[email protected]>, wrote:
>
> Here is my new bunch of remarks. In total, we are getting nearer to
> meaningful results :-)
>
> * The build process seems to be ok now, very good!
>
> * After doing `make baseline` (and `make benchmark`) I see as the last
> line
>
> ```
> Processing 100%...\nBaseline created.
> ```
>
> This `\n' looks strange.
>
> * It would be nice if `make benchmark` shows as the last line
> something like
>
> ```
> Benchmark results created in file `/foo/bar/benchmark.html`.
> ```
>
> * Looking at `benchmark.html`, I'm quite happy now with the layout,
> thanks! However, running baseline and benchmark tests for the same
> commit on my computer I'm still 'greeted' in the very first test
> line with
>
> ```
> Load 47500 0.167 0.228 -36.5
> ```
>
> 36.5% run difference is baaaad. AFAICS, you haven't yet worked on
> omitting 'warmup' iterations, right?
>
> * In `benchmark.html`, I see
>
> ```
> *Smaller values mean faster operation
> ```
>
> please insert a space after '*'. Additionally, you should mark the
> columns that are affected by this comment with '*', too, so that the
> reader knows what you are referring to.
>
>
> Werner
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.