search

mrven / Blender-Asset-Creation-Toolset

Many Tools for Game Asset Creation (Import/Export FBXs, Origin Aligment Tool, Renaming, Low-Poly Art workflow tools, etc.)
GNU General Public License v3.0
299 stars 24 forks source link

Model to look backwards. #30

Closed ani-kun closed 4 months ago

ani-kun commented 6 months ago

"Bake Axis Conversion" enabled causes the FBX model to look backwards in Unity. If "Bake Axis Conversion" is to be used, wouldn't it be better export to set "use_space_transform" to "False" and -Y axis set forward?

mrven commented 4 months ago

It's very discussion thing.

  1. Unity "Front" view looks wrong, because X-Axis direction is left, but if you enable 2D View - X-Axis direction will be right (and, I think, this is correct way) image image
  2. I already can't change this beacuse it will broke back compatibility with models exported earlier.
ani-kun commented 4 months ago

Would the appearance on the 2D View be a basis? Surely TextMesh, for example, is facing forward by default, but isn't that because of the UI?

Also, the teddy bear in the official documentation looks like a Z Forward model. https://docs.unity3d.com/ja/2023.2/Manual/GameObjects.html

Furthermore, if we are talking about compatibility, we are losing orientation compatibility (commonality) with existing fbx. If you want to overwrite old fbx with models exported with the new options, you will need to rotate the prefab as to not only change the FBX import options.

Sorry for those who are already using models exported with the "Unity2023" option, but the sooner you make the change, the better.

mrven commented 4 months ago

Ok. I will fix that and temporary add option for compatibility. And finally will remove "Unity (Legacy)" option

ani-kun commented 4 months ago

Thanks.

However, I don't see the need to remove "Unity (Legacy)" option in the future.

There may be a reason why you want to keep the Scale Factor at 1. For example, a model that has already been animated would have to be reanimated if the new Unity option was used to overwrite it.

Why would we need to remove it? This "Legacy" has great value.

mrven commented 4 months ago

I want to remove this option due to the increasing complexity of the code. I think I’ll then move this code into a separate function, which I won’t change anymore

Thank you for your feedback! 👍

mrven commented 4 months ago

Fixed in 2024.2