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.

Sign up for GitHub

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

Jump to bottom

perf: optimize getDistance #311

Open

vicb wants to merge 1 commit into manuelbieh:master from vicb:distance

vicb commented Oct 18, 2024

Calling toRad upfront reduces the number of calls from 6 to 3.

Perf test show that the updated version is a little over 1% faster. That's not much but still better.

Thanks for your great lib!


          perf: optimize getDistance

e6ef543

Calling toRad upfront reduces the number of calls from 6 to 3.

Author

vicb commented Oct 28, 2024

@manuelbieh any thoughts on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet