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

[Merged by Bors] - Faster gltf loader (re-merge of #3165) #3189

Closed
wants to merge 1 commit into from
Closed

[Merged by Bors] - Faster gltf loader (re-merge of #3165) #3189

wants to merge 1 commit into from

Commits on Nov 25, 2021

  1. Faster gltf loader (bevyengine#3165) 

    # Objective

- @superdump was having trouble with this loop in the GLTF loader.

## Solution

- Make it probably linear.
- Measured times: 
- Old: 40s, new: 200ms

I'm sure there's still room for improvement. For example, I think making the nodes be in `Arc`s could be a significant gain, since currently there's duplication all the way down the tree.

Co-authored-by: Carter Anderson <mcanders1@gmail.com>
    @DJMcNab @cart
    DJMcNab and cart committed Nov 25, 2021
    Configuration menu
    Copy the full SHA
    1d22133 View commit details
    Browse the repository at this point in the history