-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Refactor more TileCoords #10875
Refactor more TileCoords #10875
Conversation
7bd88a5
to
d9ba4db
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems ok, though I reviewed on the phone
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments.
d9ba4db
to
ccb214a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't merge this yet - I have to test some stuff regarding maximum heights that don't seem to work correctly.
ccb214a
to
1baef14
Compare
1baef14
to
1e0fc00
Compare
OK, solved the bug I encountered. |
Also converts a lot of Z calculation to use big Zs.