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

Add Kenney Character #9996

Closed
wants to merge 4 commits into from
Closed

Conversation

ameknite
Copy link
Contributor

@ameknite ameknite commented Oct 2, 2023

Objective

Suzzane was remove in #9974, I think a good replacement can be this character by Kenney.
Screenshot 2023-10-01 at 23 09 41

Solution

  • I convert the file .glb to glf with blender and change the interpolation from "STEP" to "LINEAR" to avoid this error:
    WARN bevy_gltf::loader: Animation interpolation Step is not supported, will use linear
  • Added character.gltf, character.bin and colormap.png
  • Update examples to work with the model.
  • Give credits to Kenney.
  • Remove torus.gltf

Examples

asset_loading

Screenshot 2023-10-01 at 23 11 54

hot_asset_reloading

Screenshot 2023-10-01 at 23 12 07

multiple_windows

Screenshot 2023-10-01 at 23 12 41

@alice-i-cecile alice-i-cecile added A-Assets Load files from disk to use for things like images, models, and sounds C-Examples An addition or correction to our examples and removed A-Assets Load files from disk to use for things like images, models, and sounds labels Oct 2, 2023
@mockersf
Copy link
Member

mockersf commented Oct 2, 2023

I would prefer to not change for this asset. It doesn't add to the examples where it's used, and it's larger, with more files, introducing more noise on the repo

@ameknite
Copy link
Contributor Author

ameknite commented Oct 2, 2023

@mockersf true, I could discard the animation part and it will only be 571 lines instead of 1715 in the gltf, but that might still be too much.

@alice-i-cecile
Copy link
Member

Yeah, I think the added value is too low. We'll pull in a proper humanoid model for other animation examples later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Examples An addition or correction to our examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants