`GpuComponentArrayBuffer` panics when two instances leave the array in one frame, or one comes and goes in a frame

Đang mở
  • Có gì không ổn?
#26,316 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 2 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
70/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rust
Lĩnh vực
game-dev

Hướng nghiên cứu

Start with crates/bevy_render/src/gpu_component_array_buffer.rs, especially update_components and GpuComponentArray::remove; the issue gives two reproductions and the failing assertions or command error. Trace how removals and swap-removal handle multiple entities and components added then removed before the system runs. Done means both reproductions and arbitrary same-frame removal orders complete without panicking.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

A-Rendering C-Bug D-Straightforward S-Ready-For-Implementation

Bevy version

0.20.0 (crates.io). crates/bevy_render/src/gpu_component_array_buffer.rs on main at
fd9e7609109758a2d9af401345f100be9442e9f6 (2026-10-24) is byte-identical, so main has both bugs.

What went wrong

update_components::<C> (gpu_component_array_buffer.rs:130-220) has two reachable panics.

1. A command to a despawned entity. A removed entity's entry is swap-removed: the last entry moves into the gap,
and a command inserts the moved entity's new MeshTag:

if let Some((displaced_entity, displaced_entity_new_tag)) = component_array.remove(&mut buffer, entity) {
    commands.entity(displaced_entity).insert(MeshTag::with_type::<GpuComponentArray<C>>(displaced_entity_new_tag));
}

Sometimes the displaced entity was despawned in the same frame, and its own removal comes later in
RemovedComponents. Then the command fails, and the default error handler panics:

Encountered an error in command `insert<MeshTag>`: Entity despawned: The entity with ID 544v0 is invalid; its index now has generation 1.

Common ways to remove two instances in one frame:

  • despawn a parent whose children carry the component (Children is linked_spawn);
  • despawn a batch of instances, for example on a scene or level change.

Unless the batch happens to go from the highest tag down, this panics.

2. An instance added and removed before the system ran. The component is inserted and then removed (or the entity
despawned) in the same frame, before update_components runs. Its removal is then read from RemovedComponents, but
the entity was never pushed. remove panics:

  • in a debug build: debug_assert!(!self.tag_to_entity.is_empty()) (:305) when the array is empty;
  • otherwise: .expect("Entity must have been present") (:314).

How to reproduce

use bevy::ecs::query::QueryItem;
use bevy::ecs::system::lifetimeless::Read;
use bevy::prelude::*;
use bevy::render::gpu_component_array_buffer::{GpuComponentArrayBuffer, GpuComponentArrayBufferPlugin};
use bevy::render::storage::ShaderBuffer;

#[derive(Component, Clone, Copy)]
struct Data(u32);

impl GpuComponentArrayBuffer for Data {
    type QueryData = Read<Self>;
    type QueryFilter = Changed<Self>;
    type Out = u32;

    fn extract_component(data: QueryItem<'_, '_, Self::QueryData>) -> Option<u32> {
        Some(data.0)
    }
}

fn app() -> App {
    let mut app = App::new();
    app.add_plugins((MinimalPlugins, AssetPlugin::default()))
        .init_asset::<ShaderBuffer>()
        .add_plugins(GpuComponentArrayBufferPlugin::<Data>::default());
    app
}

#[test]
fn two_instances_leaving_in_one_frame() {
    let mut app = app();
    let e: Vec<Entity> = (0..3).map(|i| app.world_mut().spawn(Data(i)).id()).collect();
    app.update(); // tags 0, 1, 2
    app.world_mut().despawn(e[0]); // its slot is filled with e[2]'s entry, and e[2] is re-tagged by a command...
    app.world_mut().despawn(e[2]); // ...but e[2] is gone too
    app.update(); // panics: `insert<MeshTag>`: Entity despawned
}

#[test]
fn an_instance_coming_and_going_in_one_frame() {
    let mut app = app();
    let e = app.world_mut().spawn(Data(0)).id();
    app.world_mut().entity_mut(e).remove::<Data>();
    app.update(); // panics in `GpuComponentArray::remove`
}

Both tests fail on bevy 0.20.0 (run 2026-10-25, a debug build without Bevy's debug feature, which names the command):

two_instances_leaving_in_one_frame: Encountered an error in command `<Enable the debug feature to see the name>`: Entity despawned: The entity with ID 33v0 is invalid; its index now has generation 1.
an_instance_coming_and_going_in_one_frame: gpu_component_array_buffer.rs:305:9: assertion failed: !self.tag_to_entity.is_empty()

Expected

Any set of entities can lose the component in one frame, in any order, and nothing panics.

Ngôn ngữ chính
Rust
Star
49.7k
Fork
5k
Merge trung bình
3 ngày 19 giờ
Pull request đã merge (30 ngày)
134

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của bevyengine/bevy

Tất cả issue của bevyengine/bevy

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.