Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bevy_gizmos crashes with MissingVertexAttributeError when rendering only a single gizmo #8432

Closed
DanielHZhang opened this issue Apr 18, 2023 · 0 comments · Fixed by #8434
Closed
Labels
A-Gizmos Visual editor and debug gizmos C-Bug An unexpected or incorrect behavior
Milestone

Comments

@DanielHZhang
Copy link
Contributor

Bevy version

Main branch

Relevant system information

AdapterInfo { name: "Apple M1 Max", vendor: 0, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Metal }

What you did

Run the 3d_gizmos example, but remove all but one of the gizmos from system:

fn system(mut gizmos: Gizmos) {
    gizmos.cuboid(
        Vec3::Y * -0.5,
        Quat::IDENTITY,
        Vec3::new(5., 1., 2.),
        Color::BLACK,
    );
}

What went wrong

Expected behaviour: Bevy renders the gizmo
Actual behaviour:

thread 'Compute Task Pool (0)' panicked at 'called `Result::unwrap()` on an `Err` value: MissingVertexAttribute(MissingVertexAttributeError { pipeline_type: Some("bevy_gizmos::pipeline_3d::GizmoPipeline"), id: MeshVertexAttributeId(0), name: "Vertex_Position" })', crates/bevy_gizmos/src/pipeline_3d.rs:154:22
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Encountered a panic in system `bevy_gizmos::pipeline_3d::queue_gizmos_3d`!
thread 'Compute Task Pool (1)' panicked at 'called `Result::unwrap()` on an `Err` value: RecvError', crates/bevy_render/src/pipelined_rendering.rs:135:45

Additional information

This seems to have been caused by #8180 which was merged in e54057c
When more than one gizmo is added, everything renders fine without crashing.

@DanielHZhang DanielHZhang added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Apr 18, 2023
@alice-i-cecile alice-i-cecile added A-Gizmos Visual editor and debug gizmos and removed S-Needs-Triage This issue needs to be labelled labels Apr 18, 2023
@alice-i-cecile alice-i-cecile added this to the 0.11 milestone Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Gizmos Visual editor and debug gizmos C-Bug An unexpected or incorrect behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants