fix: heuristic matching for "びしょ濡れのしずくちゃん" #1370
Conversation
|
@Sayamame-beans 一部ユニットテストの意図が分からなかったものもあるので確認お願いします 🙏 |
| @@ -17,7 +17,7 @@ public void TestNoPrefixSuffix() | |||
|
|
|||
| var outfit = CreateChild(root, "Outfit"); | |||
| var outfit_armature = CreateChild(outfit, "armature"); | |||
| var outfit_hips = CreateChild(outfit_armature, "hips"); | |||
| var outfit_hips = CreateChild(outfit_armature, "hip"); | |||
There was a problem hiding this comment.
元の意図: アバターがhipで衣装がhipsでもsuffixは空であることの確認
There was a problem hiding this comment.
変更されているようにhip同士でも十分そう
(TestDifferentHipsNameにおける"pre_hips.suf"があれば十分そう)
| @@ -42,21 +42,33 @@ public void TestDifferentHipsName() | |||
|
|
|||
| var outfit = CreateChild(root, "Outfit"); | |||
| var outfit_armature = CreateChild(outfit, "armature"); | |||
| var outfit_hips = CreateChild(outfit_armature, "pre_Hips.suf"); | |||
| var outfit_hips = CreateChild(outfit_armature, "pre_hips.suf"); | |||
There was a problem hiding this comment.
元の意図: ケース違いでの対応チェック(ケース違いにする必要なさそうかも)
| @@ -101,8 +115,38 @@ public void TestSameHipsName_Fail() | |||
|
|
|||
| outfit_mama.InferPrefixSuffix(); | |||
|
|
|||
| // Current(v1.10.x) InferPrefixSuffix fail to infer prefix/suffix when avatar has unique prefix/suffix and outfit has their name | |||
There was a problem hiding this comment.
元の意図: 完全一致の優先度が低かったため、hipsとしての認識が優先されていた(多分そう)
There was a problem hiding this comment.
bd_さんがされてるように一致することが理想的で、しかしあの段階では無理であったので、無理であることの確認が入ってました
There was a problem hiding this comment.
今回の変更により問題なくなった(推定出来るようになった)
👍
There was a problem hiding this comment.
確認: 旧手法(アバター側ボーン名との一致確認)をやり、その後に新手法(辞書確認)をやり、ボーン全体での一致数を見た際に後者が前者の2倍以上一致しているなら後者を採用、それ以外は前者
特に恩恵を受けるのは、normalize処理をしない純粋な一致確認でも十分にprefix/suffixが推定出来る(しずくさんとその対応衣装のような)ケース
良さそうですね…?
| @@ -17,7 +17,7 @@ public void TestNoPrefixSuffix() | |||
|
|
|||
| var outfit = CreateChild(root, "Outfit"); | |||
| var outfit_armature = CreateChild(outfit, "armature"); | |||
| var outfit_hips = CreateChild(outfit_armature, "hips"); | |||
| var outfit_hips = CreateChild(outfit_armature, "hip"); | |||
There was a problem hiding this comment.
変更されているようにhip同士でも十分そう
(TestDifferentHipsNameにおける"pre_hips.suf"があれば十分そう)
| @@ -101,8 +115,38 @@ public void TestSameHipsName_Fail() | |||
|
|
|||
| outfit_mama.InferPrefixSuffix(); | |||
|
|
|||
| // Current(v1.10.x) InferPrefixSuffix fail to infer prefix/suffix when avatar has unique prefix/suffix and outfit has their name | |||
There was a problem hiding this comment.
今回の変更により問題なくなった(推定出来るようになった)
👍
| } | ||
|
|
||
| [Test] | ||
| public void TestSpuriousMatch() |
No description provided.