| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
| |
|
|
|
|
| |
This is done in order to improve labels stability and for the performace reasons.
|
|
|
|
|
|
| |
If the view is not tilted, we want the new symbols to show up faster, so we squeeze the placement period.
On contrary, with the tilted view it's more important to make placement rarely for performance reasons and as the new symbols are normally "far away" and the user is not that interested to see them ASAP.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
The `symbolIntersectsTileEdges` algorithm should not be applied to the symbols with `line-center` placement
|
| |
|
|
|
|
|
|
| |
Tile mode placement algorithm now checks if bounding boxes for both label text and icon are intersecting the edges of the tiles.
Before, it checked only text bounding boxes and thus label icons might have got cut off.
|
| |
|
|
|
|
| |
operator
|
| |
|
| |
|
|
|
|
|
|
|
| |
Map and renderer / orchestrator should be able to run on a separate threads,
however, legacy AnnotationManager is shared between Map and Renderer, therefore
is not a thread safe. Until AnnotationManager is deprecated and removed from a
code-base, use it only via weak pointers.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
It triggers Metal depth stencil validation error since depth test is enabled with no depth attachment.
|
|
|
|
| |
Fix using of invalid iterators.
|
|
|
|
|
|
| |
They have a different license from the rest of the code
and this will make it explicit. Also the script for generating
the license will dectect them correctly.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
and get rid of `SpriteLoaderWorker`, use `scheduleAndReplyValue()` API instead.
|
| |
|
| |
|
|
|
|
| |
This reverts commit b5c386ad7708ca9fe14f219b0ffc94cf6ce8fc88.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
If we get a new sprite sheet from the server, we need to merge current
sprite sheet with a new one, while overwiting duplicates and keeping
old unique images in a style.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Implement in.cpp
* Fix review comments.
* Add expression_equality test for 'in'
* Fix review comments.
* [core] Update changelog.
* [core] Update mapbox-gl-js
* [core] Ignore render-tests/debug/padding
* [core] Update baseline.
|
|
|
|
|
|
|
| |
We used some shared SegmentVectors, e.g. for drawing raster or background tiles.
In longer running maps, this lead to resource accumulation. By storing the SegmentVectors
and the contained gfx::DrawScope objects, we ensure that resources get released
when the associated render objects vanish.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Fix artifact for zero-lenghted dash array
Fixes issue https://github.com/mapbox/mapbox-gl-js/issues/9213 and add
render test for coverage around 0-length dash arrays
Update distance field generation for regular dashes as follows:
1. Compute the dash array ranges and stretch location along the distance field
2. Collapse any 0-length distance field range
3. Collapse neighbouring same-type parts into single part
* combine consecutive dashes and parts
https://github.com/mapbox/mapbox-gl-js/pull/9246
https://github.com/mapbox/mapbox-gl-native/issues/16181
|
|
|
|
|
|
| |
In cases when cached resource is useable, yet don't have an expiration
timestamp, we provided data to the requester from the cache and the same
data was returned once 304 response was received from the network.
|
|
|
|
| |
This reverts commit 80cb05420a86ed53815cae7fb2cb3fddf07dd1d1.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
What happened:
The excessive `onSpriteLoaded()` notifications were caused by the trick inside `OnlineFileRequest::completed()` implemenation,
which causes receving of two file source updates pointing to the *same* data in case the network response is `notModified` and
the local storage has the sprite sheet already stored inside.
In the SpriteLoader` implementation, it resulted in 2 extra `parseSprite()` calls and 2 extra `onSpriteLoaded()` notifications.
Then, the excessive `onSpriteLoaded()` notifications affected style images and all the unnecessarily added image dupes (btw,
the amount of these dupes `= 2 * number of style images`) were treated by the `RenderOrchestrator` as *updated* images.
Further, `RenderOrchestrator` called `ImageManager::updateImage()` for the updated images and the `ImageManager` populated the
`updatedImageVersions` map.
Finally, at every frame all the geometry tiles iterated through the `updatedImageVersions` to check whether their uploaded
textures needed patching.
All the mentioned above had a negative performance impact.
The fix:
Now, `SpriteLoader` sends `onSpriteLoaded()` notifications only if the data in the file source response is different comparing
to the existing one.
|
| |
|
|
|
|
| |
Address review findings/nit
|
|
|
|
|
|
|
| |
Fix polygon within algorithm
Add Unit tests
Fix incorrect metrics folder for ios-render-test-runner job
|
|
|
|
|
|
| |
add canonical as pointer
fix review findings
|
|
|
|
| |
Add geometry type checker
|