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

scene_viewer panics when given a gltf with a camera #11905

Closed
rparrett opened this issue Feb 16, 2024 · 1 comment · Fixed by #11907
Closed

scene_viewer panics when given a gltf with a camera #11905

rparrett opened this issue Feb 16, 2024 · 1 comment · Fixed by #11907
Labels
A-Assets Load files from disk to use for things like images, models, and sounds A-Rendering Drawing game state to the screen C-Bug An unexpected or incorrect behavior
Milestone

Comments

@rparrett
Copy link
Contributor

rparrett commented Feb 16, 2024

Bevy version

main 80f2ee2

Possibly broken by #11868 adding exposure settings to camera bundles?

Relevant system information

AdapterInfo { name: "Apple M1 Max", vendor: 0, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Metal }
SystemInfo { os: "MacOS 14.2.1 ", kernel: "23.2.0", cpu: "Apple M1 Max", core_count: "10", memory: "64.0 GiB" }

What you did

Exported a gltf (bistro outdoor exported as gltf in blender with camera export enabled)

cargo run --release --example scene_viewer -- ~/untitled.glb

What went wrong

2024-02-16T18:09:57.052459Z  INFO scene_viewer::scene_viewer_plugin: Spawning scene...
thread 'main' panicked at crates/bevy_scene/src/scene_spawner.rs:431:35:
scene contains the unregistered type `bevy_render::camera::camera::Exposure`. consider reflecting it with `#[derive(Reflect)]` and registering the type using `app.register_type::<T>()`
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Encountered a panic in exclusive system `bevy_scene::scene_spawner::scene_spawner_system`!
Encountered a panic in system `bevy_app::main_schedule::Main::run_main`!
thread 'main' panicked at /Users/me/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.29.10/src/platform_impl/macos/app_state.rs:387:33:
called `Result::unwrap()` on an `Err` value: PoisonError { .. }
@rparrett rparrett added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Feb 16, 2024
@alice-i-cecile
Copy link
Member

This is an extension of #11852. @cart, your call on if we block on this, or put a fix in a point release.

@alice-i-cecile alice-i-cecile added A-Rendering Drawing game state to the screen A-Assets Load files from disk to use for things like images, models, and sounds and removed S-Needs-Triage This issue needs to be labelled labels Feb 16, 2024
@alice-i-cecile alice-i-cecile added this to the 0.13 milestone Feb 16, 2024
github-merge-queue bot pushed a commit that referenced this issue Feb 16, 2024
# Objective

- Don't crash when loading a scene with a camera

## Solution

- Derive Reflect for Exposure

Closes #11905
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Assets Load files from disk to use for things like images, models, and sounds A-Rendering Drawing game state to the screen C-Bug An unexpected or incorrect behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants