Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[VRM] T&OのAutoMergeSkinnedMeshがVRMのFirstPerson性を考慮しない #1071

Open
kaikoga opened this issue May 12, 2024 · 1 comment · May be fixed by #1103
Open

[VRM] T&OのAutoMergeSkinnedMeshがVRMのFirstPerson性を考慮しない #1071

kaikoga opened this issue May 12, 2024 · 1 comment · May be fixed by #1103
Labels
bug Something isn't working p:VRM VRM Platform

Comments

@kaikoga
Copy link
Contributor

kaikoga commented May 12, 2024

でまとめてしまうので、FirstPerson性が変化してしまう。
例えば、FirstPerson = Third Person Onlyに設定した前髪メッシュがFirstPerson = Bothに統合されることにより、FirstPerson = Bothの見た目に変化してしまう（この場合は、一人称視点からも見えるようになる）。

補足説明：メッシュのFirstPerson性はVRMコンポーネント（ VRM First Person / VRMInstance に設定されたVRM10Object ）が定義します

すぐには直せないと思うので、将来的にndmfがVRMに正式対応した時のために暫定Workaroundを書いておきます。

Workaround

以下のどちらかをやる

  • T&O のAuto Merge Skinned Meshをオフにするか、Merge Skinned Meshを全て手動で設定することでAuto Merge Skinned Meshを無効にする
  • T&O のAuto Merge Skinned Meshをオンにする場合は、まとめたくないメッシュ（例えば、VRMのThird Person Onlyに設定したメッシュ）だけをまとめるMerge Skinned Meshを手動で設定しておく
    • （することで何故か大丈夫でしたが、原理は理解しきれていません）
    • Auto Merge Skinned Meshでまとめて欲しいメッシュは無設定でOK
@anatawa12 anatawa12 added bug Something isn't working p:VRM VRM Platform labels May 12, 2024
@anatawa12
Copy link
Owner

anatawa12 commented May 12, 2024 •
edited
Loading

２つ目のworkaroundはMergeSMRのroot bone/bounds等を正しく設定すると多分マージされるようになりますね

修正はFirstParson性をCategorizationKeyに追加するのが正しそう。

AvatarOptimizer/Editor/Processors/TraceAndOptimize/AutoMergeSkinnedMesh.cs

Line 647 in 5bd73da

private struct CategorizationKey : IEquatable<CategorizationKey>

kaikoga added a commit to kaikoga/AvatarOptimizer that referenced this issue Jun 23, 2024
@kaikoga
fix(T&O): also categorize SMRs by VRM FirstPerson flags (fix anatawa1…
e4b70de 
…2#1071)
@kaikoga kaikoga linked a pull request Jun 23, 2024 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working p:VRM VRM Platform
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(T&O): VRMのFirstPersonFlagをCategorizationKeyに追加する kaikoga/AvatarOptimizer
2 participants
@kaikoga @anatawa12