Georges Hatem discussion

[gcarreno]

Hey Georges(@georges-hatem),

This kicks off the discussion about your entry.

Which, unfortunately is now stuck due to an issue with mORMot2 not compiling on Delphi once you set a Linux 64 target.

Already dropped an issue on the mORMot2 repository: synopse/mORMot2#246

Let's see what happens.

Cheers,
Gus

[synopse]

mORMot does not support Delphi for Linux.
We use FPC to compile for Linux - but the code needs to be FPC and Delphi compatible.

[gcarreno]

Hey Georges(@georges-hatem),

Last PR hash is good:

$ ./test_all.sh ghatem
******** Test ********

===== Georges Hatem - FPC ======

real	0m5.987s
user	2m26.719s
sys	0m2.403s
4256d19d3e134d79cc6f160d428a1d859ce961167bd01ca528daca8705163910  /home/gcarreno/Programming/1brc-ObjectPascal/results/ghatem.output
4256d19d3e134d79cc6f160d428a1d859ce961167bd01ca528daca8705163910  Official Output Hash
===========

Timing with Desktop on:

$ ./run_all.sh ghatem
******** Run ********

===== Georges Hatem - FPC ======
-- SSD --
Benchmark 1: ghatem
  Time (mean ± σ):      6.016 s ±  0.082 s    [User: 145.823 s, System: 2.436 s]
  Range (min … max):    5.880 s …  6.168 s    10 runs
 
===========

hyperfine file:
ghatem-1_000_000_000-SSD.json

[gcarreno]

Hey Georges(@georges-hatem),

Latest PR results on a not so quiet system:

$ ./run_all.sh ghatem
******** Run ********

===== Georges Hatem - FPC ======
-- SSD --
Benchmark 1: ghatem
  Time (mean ± σ):      4.191 s ±  0.054 s    [User: 99.295 s, System: 2.275 s]
  Range (min … max):    4.120 s …  4.263 s    10 runs
 
===========

And the latest hyperfine file:
ghatem-1_000_000_000-SSD.json

[georges-hatem]

thanks Gus for the separate results. that's more improvements than expected

[paweld]

Hi @georges-hatem
Your latest version compiled with the Release configuration (Lazarus trunk / FPC trunk) causes an error: ERROR: Stream write error

[georges-hatem]

Hello @paweld ,
Right because the 3 configurations to compile are Hashmod, Hashmult and Lemire.

If you prefer that I set it up differently please let me know.

[paweld]

It doesn't bother me, but I know that @gcarreno compiles only the release configuration so your version won't pass the tests.

[georges-hatem]

Got it. I'll unify it all back under Release once the winner of the 3 becomes clear