[openstreetmap/openstreetmap-website] Change zoom precision function (PR #5084)
Tom Hughes
notifications at github.com
Fri Aug 16 17:05:59 UTC 2024
> `ceil( log10(2**(8+zoom) / 180) )`
Would it not be better just to use that then? Yes I can prove using logarthmic identities that what you've written is the same but it's not clear why you prefer the transformed version?
I also suspect the `Math.max` is no longer needed as it can't go negative for any sane zoom - it was needed before because log10 breaks down for zoom zero...
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5084#issuecomment-2293856827
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5084/c2293856827 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240816/ab093d22/attachment.htm>
More information about the rails-dev
mailing list