2016/10/21 に行われた 『第 6 回 UE4 よちよち勉強会』において使用したスライドです。

1. UE4 の『 Bone の回転バグ』を直して
プルリク投げた話
らりほま
2016/10/21 第 6 回 UE4 よちよち勉強会

2. 2
Boneの回転バグとは？

3. 3
https://issues.unrealengine.com/issue/UE-15388

4. 4
バグの概要
●
FBX を
●
Skeletal Mesh として Import する際に
●
Scale を変更すると
●
Persona で Bone を回したときに
●
回転量がおかしくなる

5. 5
バグの概要
●
FBX を

6. 6
バグの概要
●
Skeletal Mesh として Import する際に

7. 7
バグの概要
●
Scale を変更すると

8. 8
バグの概要
●
Persona で Bone を回したときに

9. 9
バグの概要
●
回転量がおかしくなる

10. 10
バグの概要
●
回転量がおかしくなる

11. 11
バグの概要
●
Scale を増加させるほど回転量が減少する
●
ぶっちゃけそんなに困るバグではない
– Blueprint での回転処理は正常
– 困るのはリターゲット基底ポーズ調整時のみ
●
でも気になる
– UE4.7 のときには既に存在

12. 12
直そう！

13. 13
エンジンの
ソースコードのどこで
回転処理を行っている？

14. 14
それっぽいところに
ブレークポイントを
仕掛けて探そう！

15. 15
エンジンデバッグの注意点
●
ソリューション構成を
Debug Editor にしてビルドする

16. 16
エンジンデバッグの注意点
●
ブレークポイントは
プロジェクトのソリューションに置く
( エンジンのソリューションではない )

17. 17
エンジンデバッグの注意点
エンジン
プロジェクト ( ブレークポイントはこっちに置く )
( ここでエンジンの起動が終了 )

18. 18
エンジンデバッグの注意点
●
参考： alwei さんのブログ記事
– UE4 VisualStudio を使ってエンジンをデバッグする -
Let's Enjoy Unreal Engine
http://unrealengine.hatenablog.com/entry/2
014/08/22/215802

19. 19
回転処理を行う
コードを見つけた！

20. 20
問題の回転処理
●
Engine/Source/Editor/Persona/Private/EditModes
/SkeletonSelectionEditMode.cpp

21. 21
Scaleに応じて
回転量が変化する
というバグなので…

22. 22
Scaleを考慮した計算に
なってしまっている
部分を探そう！

23. 23
問題の回転処理
●
Engine/Source/Editor/Persona/Private/EditModes
/SkeletonSelectionEditMode.cpp

24. 24
問題の回転処理
●
EngineSourceRuntimeCorePublicMathTransf
ormVectorized.h

25. 25
Scaleを考慮しない
計算にするには…

26. 26
問題の回転処理
●
EngineSourceRuntimeCorePublicMathTransf
ormVectorized.h
– TransformVector 定義の下に
TransformVectorNoScale
( Scale を考慮しない版 ) の定義が存在

27. 27
TransformVector
↓
TransformVectorNoScale

28. 28
たった 7 文字 (“Noscale”)
追加するだけで (たぶん) 解決！
ボキッ

29. 29
pull request の流れ
●
fork→clone→pull request 用 blanch 作成
→ ( 修正 )→add→commit→push
→pull request

30. 30
取り込まれるといいな…
ここまで書いておいて reject されたら恥ずかしい (*ﾉ▽ﾉ)