Remove old per-item rendering pipeline and consolidate into GPU-driven path#141
Remove old per-item rendering pipeline and consolidate into GPU-driven path#141
Conversation
|
Can completely remove the old per-item pipeline, and only keep the Merged ones (the GPU-driven one), and rename them to name without |
Co-authored-by: AzurIce <22115534+AzurIce@users.noreply.github.com>
Done in c247f0a. Removed all old per-item pipeline nodes ( |
The GPU-driven renderer was still performing a per-item
alloc_packetcall for everyVItemon each frame, and the old CPU-driven per-item rendering pipeline was never removed when the merged (GPU-driven) rendering path was introduced.Changes
VItempool.alloc_packetloop inrender_store_with_pool(packages/ranim-render/src/lib.rs)VItemComputeNode,VItemDepthNode,VItemColorNode(per-item path)VItemColorPipeline,VItemDepthPipeline,VItemComputePipeline(per-item path) and associated bind groupsvitem.wgsl,vitem_compute.wgsl(per-item path)VItemRenderInstance,PlaneUniform, andimpl Primitive for VItemfromprimitives/vitem.rsMerged*) nodes and pipelines to drop theMergedprefix:MergedVItemComputeNode→VItemComputeNodeMergedVItemDepthNode→VItemDepthNodeMergedVItemColorNode→VItemColorNodeMergedVItemComputePipeline→VItemComputePipelineMergedVItemColorPipeline→VItemColorPipelineMergedVItemDepthPipeline→VItemDepthPipelinemerged_vitem.wgsl→vitem.wgsl,merged_vitem_compute.wgsl→vitem_compute.wgslbuild_merged_render_graph()→build_render_graph()and removed the deprecated oldbuild_render_graph()inlib.rsThe GPU-driven path (now the only path) reads exclusively from
MergedVItemBuffer, issuing a single batched draw call for all items rather than allocating and dispatching per-item GPU resources.Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.