-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
flexpolyline bench #46
flexpolyline bench #46
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.