Conversation
e92c541 to
a3a4028
Compare
a3a4028 to
fd9011d
Compare
|
This might be an interesting test case @urschrei. Before #48, polyline encode was faster for me than flexpolyline. Now it's slower: Edit: this is on M1 aarch64 Decode is still faster (actually even faster than it was before!) Edit: on x86_64 polyline is still a little faster than flexpolyline for both encode and decode |
|
So if I'm reading this correctly #48 is still giving me across-the-board improvements, we're faster than flexpolyline, and the flexpolyline tests have some noise (up to 3 %). |
|
It sounds like you live in the kind of sane world I would like to live in. Not sure if you saw my edit above, but I saw behavior similar to yours on my x86/64 machine. I'm becoming more and more convinced that I am inhabiting some weird M1 edge case, and we shouldn't spend more time trying to figure it out. |
|
And after rebasing against #49: another small improvement on my machine. |
|
I am highly confused. I think we should merge this (because I think the flexpolyline comparisons are valuable), and solicit benchmarking runs from some other people with M1/2/3 machines to see whether anything emerges. |
update: merged!
Based on #45, so merge that first (Sorry for the long chain of PR's. )FIXES #35
(actually it just demonstrates that it's been addressed by previous work by me and @mattiZed)
flexpolyline is 20% slower for encode
flexpolyline is 12% slower for decode
These aren't very rigorous benchmarks. In fact, I'd be OK with not merging them at all, but it might be a good thing to keep around in case we break something and become 10x slower than flexpolyline.