antvis / L7

🌎 Large-scale WebGL-powered Geospatial Data Visualization analysis engine.
https://l7.antv.antgroup.com
MIT License
3.56k stars 625 forks source link

fix: 修复 bindings 缓存在 WebGPU 的问题 #2526

Closed lvisei closed 3 weeks ago

lvisei commented 3 weeks ago

🤔 这个变动的性质是?

🔗 相关 Issue

💡 问题出现原因

在 WebGPU 下,每次 model draw 创建新 Pipeline,使用了缓存 createBindings,导致新的 Pipeline 使用 getBindGroupLayout 不一致。

因 L7 API 设计原因,draw 动态传入的参数(比如 stencil),必须每次创建 Pipeline,后续设计新 API 考虑 Pipeline 缓存情况。

changeset-bot[bot] commented 3 weeks ago

🦋 Changeset detected

Latest commit: 8f12ee2f99690156ac53bf69006b95c9f93ee1d3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 12 packages | Name | Type | | ------------------- | ----- | | @antv/l7-renderer | Patch | | @antv/l7-scene | Patch | | @antv/l7 | Patch | | @antv/l7-test-utils | Patch | | @antv/l7-three | Patch | | @antv/l7-component | Patch | | @antv/l7-layers | Patch | | @antv/l7-core | Patch | | @antv/l7-map | Patch | | @antv/l7-maps | Patch | | @antv/l7-source | Patch | | @antv/l7-utils | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR