| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
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.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
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
|
|
|
|
| |
fix #15964
partially port mapbox/mapbox-gl-js#9054
|
| |
|
|
|
|
|
|
| |
This change enables attaching images to the style with batches and
avoids massive re-allocations. Thus, it improves UI performance
especially at start-up time.
|
|
|
|
| |
Backport GL-JS fix 'Hide glyphs behind the camera'(https://github.com/mapbox/mapbox-gl-js/pull/9229)
|
|
|
|
|
|
|
| |
change
Clear tile pyramid only if updated source has different tile options,
zoom range or initialization state for a custom tile loader.
|
| |
|
| |
|
|
|
|
|
|
|
| |
New setPrefetchZoomDelta(optional<uint8_t> delta) method allow overriding
default tile prefetch setting that is defined by the Map instance. The
method can be moved to generic style specification if found to be useful
for gl-js engine.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change fixes the following problems:
1) Before this change, when multiple symbols were grouped close to the
tile borders, different tiles could place different symbols, because each
tile arbitrary assigned feature ids, and these ids defined the placement
order being applied. This caused artifacts at the tile boundaries.
With this change, in tile mode the placement order of two symbols crossing
a tile border is defined by their anchor `y` values. It means that the
symbols are being placed the same order across all the tiles.
2) Before this change, the engine did not place with priority the symbols
crossing the borders between two neighboring tiles. Now it does, and it
improves a lot symbol placement stability in the tile map mode.
|
|
|
|
|
|
| |
* [build] Adding a build flag to build without GL if needed
* [build] Moving custom layer to mbgl/gl
|
|
|
|
|
| |
New method can be used for aggregating property expressions from
multiple properties(layers) into single match / case expression.
|