Skip to content
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 label collider tile scale #2142

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rwrx
Copy link
Contributor

@rwrx rwrx commented Feb 1, 2020

While I was testing tile fallbacking feature I discovered that when tile fallback (there is a pull request for tile fallback here: #2120) is from zoom 16 to zoom 0 there just occurred an int 32 overflow in allocation https://github.com/tangrams/isect2d/blob/2e1a75cee09d9949900e926c61c86505b09205b2/include/isect2d.h#L60 . So while trying it and profiling memory usage on Android device I came out with a solution to not process labels when tile scale difference is larger or equal to 6 so it won't much consume RAM and also it won't crash on bad alloc exception. I am not sure if this proposed solution is a right one.

@tallytalwar
Copy link
Member

@hjanetzek or @karimnaaji would remember the code here the most. But I can give it a try. Thanks @rwrx

Base automatically changed from master to main February 15, 2021 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants