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.

Sign up for GitHub

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

Jump to bottom

GLTF loading and animation performance regression #17535

Open
aloucks opened this issue Jan 25, 2025 · 2 comments
Open

GLTF loading and animation performance regression #17535

aloucks opened this issue Jan 25, 2025 · 2 comments
Labels
A-Animation Make things move and change over time C-Bug An unexpected or incorrect behavior C-Performance A change motivated by improving speed, memory usage or compile times P-Regression Functionality that used to work but no longer does. Add a test for this! S-Needs-Investigation This issue requires detective work to figure out what's going wrong

Comments

@aloucks
Copy link
Contributor

aloucks commented Jan 25, 2025

Bevy version

main
deb135c

Relevant system information

$ cargo --version -vvv
cargo 1.83.0 (5ffbef321 2024-10-29)
release: 1.83.0
commit-hash: 5ffbef3211a8c378857905775a15c5b32a174d3b
commit-date: 2024-10-29
host: x86_64-pc-windows-msvc
libgit2: 1.8.1 (sys:0.19.0 vendored)
libcurl: 8.9.0-DEV (sys:0.4.74+curl-8.9.0 vendored ssl:Schannel)
os: Windows 10.0.22631 (Windows 11 Professional) [64-bit]

What you did

cargo run --example scene_viewer --features animation assets/models/UndeadMale/scourgemale.gltf

What went wrong

The gltf file never finishes loading when running with the dev profile (or at least it takes a very very long time - I've killed the process after a few minutes). It will eventually load when running it with --release but it's slow and the animation playback is very choppy.

Additional information

The file loads nearly instantly with v0.15.1 and animation playback is smooth, even with the dev profile.

UndeadMale.zip
@aloucks aloucks added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Jan 25, 2025
@rparrett rparrett added C-Performance A change motivated by improving speed, memory usage or compile times P-Regression Functionality that used to work but no longer does. Add a test for this! A-Animation Make things move and change over time S-Needs-Investigation This issue requires detective work to figure out what's going wrong and removed S-Needs-Triage This issue needs to be labelled labels Jan 25, 2025
@rparrett
Copy link
Contributor

Are you able to bisect?

@aloucks
Copy link
Contributor Author

aloucks commented Jan 25, 2025

Are you able to bisect?

The first bad commit is 21f1e30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Animation Make things move and change over time C-Bug An unexpected or incorrect behavior C-Performance A change motivated by improving speed, memory usage or compile times P-Regression Functionality that used to work but no longer does. Add a test for this! S-Needs-Investigation This issue requires detective work to figure out what's going wrong
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rparrett @aloucks