GTLF importer doesn't set Root Motion to true when animation type is mecanim (non-humanoid)

[XtroTheArctic]

Describe the bug 💬

This behavior is inconsistent with FBX importer and requires user to apply an prefab override manually on every single model imported.

FBX Importer (!!! IMPORTANT !!!! For this to work, Root node in "Rig" tab must be assigned correctly):

UnityGLTF Importer:

Steps to reproduce 🔢

1. Import a model with "Generic" animation type.
2. See that Root Motion is unchecked.

Files to reproduce the issue ♻

Model files are added in the comment section.

Editor Version 🎲

Other (Provide details below)

Render Pipeline and version

Not relevant

UnityGLTF Version

2.17.1

Operating System 👩‍💻

Windows

When does this problem happen?

• Editor Import
• Runtime Import
• Editor Export
• Runtime Export

Additional Info 📜

No response

Validations 🩹

• I have searched existing issues: no issue already exist that reports the same problem.
• I follow the Code of Conduct
• I provided a minimal reproducible example, including files when necessary.

[XtroTheArctic]

FBX and GLB files to repro the problem.
Horse.zip

[XtroTheArctic]

Earlier (20 hours ago), In the PR for this issue, I commented that I found a problem with that PR. That was wrong. I wrote that comment at a very late hour and I wasn't careful enough.

Now I am back for more investigation results.

The blender model in this zip is a building and it doesn't have any animation or rigging related stuff. It's just a pure mesh model.
Bakery.zip

On this tab of Blender, you can export it as glb or fbx if you want but I already included the exported FBX file in the zip.

Even though, the model is not rigged and animated, Unity's FBX importer is creating an animator on it with the root motion property set to true but you must select a Root node first.

Unity's FBX importer doesn't need the model to contain an animation to be able to add an animator with root motion set to true.

Since there is no root node selection in UnityGLTF importer, I understand that UnityGLTF relies on minimum one animation in the model and that's OK, I guess but still, the animator created for that animated model needs to have "apply root motion" set to true.

As a result, my PR is still valid. I'd be happy if you could check it and merge it soon.

Thank you very much for everything.

[XtroTheArctic]

PR 870 is not valid anymore. Please see my comment there.

This issue will be fixed along with #872 in a new PR.

[XtroTheArctic]

PR submitted: #877