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

Load scene data from FBX tree #27

Merged
merged 9 commits into from
Aug 4, 2022
Merged

Load scene data from FBX tree #27

merged 9 commits into from
Aug 4, 2022

Conversation

nicopap
Copy link
Owner

@nicopap nicopap commented Jul 29, 2022

Read the FBX transform tree and assign entities for each node in the hierarchy,
computing and setting the bevy Transform according to FBX's completely
insane transform propagation algorithm.

Also a few unrelated things (I'm sorry)

This is also a first step toward loading skeleton rigs.

  • Correct the way the scene's UnitScaleFactor is applied: instead of dividing
    the vertices position by it, we just add the scale to the root node of the scene,
    this also requires dividing by default by 100.0 the scale of all FBX files.
  • Add a tool to print the FBX file content in the dev_docs directory
  • Add a "Contributing" section to the README
  • Fixes Load the scene tree #26
  • The Bistro scene's normals are broken, but this is a bevy bug: Lighting issue on scaled objects after upgrading to 0.8 bevyengine/bevy#5514

nicopap added 6 commits August 2, 2022 10:39
Read the scene tree and assign entities for each node in the hierarchy,
assigning FbxMesh to their proper place in it.

The current implementation still has major issues,
visible in the Paris Bistro scene from Amazon research.

For example the sewer covers, one of the plant pots, the bistro door
and the light garlands are not moved properly to their right place.

However, now the scene is the right side up, which makes it much easier
to deal with.
@nicopap nicopap marked this pull request as ready for review August 4, 2022 15:59
@nicopap nicopap merged commit d00b15f into nicopap:main Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Load the scene tree
1 participant