Page MenuHome
Differential D15035

Fix FBX subdivision BoundaryRule
ClosedPublic
Actions

Authored by Samuli Raivio (bqqbarbhg) on May 26 2022, 6:03 PM.

Details

Reviewers
Bastien Montagne (mont29)
Commits
rBA7a8502871c34: Fix FBX IO handling of BoundaryRule enum values.
Summary

I submitted D12204 some time ago that writes BoundaryRule to FBX exports for subdivision surfaces (if enabled). Turns out I got the enumeration value the wrong way around so Blender import/export is consistent but it may be inconsistent with other software. This patch fixes the issue and makes the values consistent with the FBX format.

Diff Detail

Event Timeline

Samuli Raivio (bqqbarbhg) requested review of this revision.May 26 2022, 6:03 PM
Samuli Raivio (bqqbarbhg) created this revision.
Pratik Borhade (PratikPB2123) added subscribers: Bastien Montagne (mont29), Pratik Borhade (PratikPB2123).Jul 11 2022, 3:27 AM

Hi, thanks for the patch
Ask @Bastien Montagne (mont29) for review :)

Bastien Montagne (mont29) accepted this revision.Jul 11 2022, 11:51 AM
This revision is now accepted and ready to land.Jul 11 2022, 11:51 AM
Closed by commit rBA7a8502871c34: Fix FBX IO handling of BoundaryRule enum values. (authored by Samuli Raivio (bqqbarbhg), committed by Bastien Montagne (mont29)). · Explain WhyJul 11 2022, 12:02 PM
This revision was automatically updated to reflect the committed changes.
Bastien Montagne (mont29) added a commit: rBA7a8502871c34: Fix FBX IO handling of BoundaryRule enum values..

Revision Contents
Changeset List

PathSize
io_scene_fbx/
2 lines
4 lines
2 lines

Diff 51873

View Options

io_scene_fbx/__init__.py

Loading...
View Options

io_scene_fbx/export_fbx_bin.py

Loading...
View Options

io_scene_fbx/import_fbx.py

Loading...