-
-
Notifications
You must be signed in to change notification settings - Fork 944
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
fix: deterministic results for address.nearbyGPSCoordinate #737
fix: deterministic results for address.nearbyGPSCoordinate #737
Conversation
Codecov Report
@@ Coverage Diff @@
## main #737 +/- ##
=======================================
Coverage 99.34% 99.34%
=======================================
Files 1925 1925
Lines 177024 177030 +6
Branches 909 909
=======================================
+ Hits 175871 175877 +6
Misses 1097 1097
Partials 56 56
|
@xDivisionByZerox Tests need update (?) |
I'm testing local again to see if I forgot something, but I normally always run tests locally before pushing :/ |
The |
Well, that's funny;
But if I run the test suite it gets |
Should we use 1e-5 ? |
I think 0.01 is enough. EDIT: Let me do some quick math. |
I'll just do that. With 4 decimal points, it just works sometimes and sometimes it doesn't. |
Let's see if this works one last time before I update the precision. |
🎉🎉🎉 |
closes #736