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.

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

[1.0] Expression の override と binary の組み合わせの挙動を実装 #2464

Merged
merged 10 commits into from
Oct 24, 2024

Conversation

ousttrue
Copy link
Contributor

@ousttrue ousttrue commented Oct 16, 2024

fixed #2430

vrm-c/vrm-specification#487

および Assets/VRM10_Samples/VRM10Viewer に expression 実験用の UI を追加しました

viewer_expression_ui

@ousttrue ousttrue changed the title [WIP] expression [1.0] Expression の override と binary の組み合わせの挙動を実装 Oct 21, 2024
if (blinkMultiplier < 1.0f)
{
actualWeights[key] = 0.0f;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isBinaryが指定されている表情が他の表情からオーバーライドの影響を受ける場合、受けている影響が0.0より大きければ、完全に抑制されなければいけません(MUST)。

@ousttrue
Copy link
Contributor Author

isBinaryが指定されている表情が他の表情にオーバーライドの影響を与える場合、出力値である二値化された値をもって他の表情に影響を与えなければいけません(MUST)。

に関しては既存実装で実現できているように見えたので特に変更していません。

isBinaryが指定されている表情が他の表情からオーバーライドの影響を受ける場合、受けている影響が0.0より大きければ、完全に抑制されなければいけません(MUST)。

を実装しました。

@s-iwaki-d s-iwaki-d requested review from 0b5vr and removed request for Santarh October 22, 2024 08:25
@0b5vr
Copy link
Contributor

0b5vr commented Oct 24, 2024

isBinaryの表情がoverrideを行う場合ならびにisBinaryの表情がoverrideされる場合のサンプルモデルを作成しました。

image

isbinary-overrides.zip

UniVRMにおいて、同じaccessorが複数のメッシュで用いられている場合にUVのY-flipがうまく動作しないことがある問題があり、現状本モデル群は意図通りに動作していません。
isBinaryとoverrideの挙動については、本PRをもって良好に動いていそうです。
UVの問題については別途issueを建てさせていただきます。

image

Copy link
Contributor

@0b5vr 0b5vr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

サンプルコードはしっかりとは見ていませんが、少なくとも DefaultExpressionValidator 内のロジックは良さそうです。

@ousttrue ousttrue merged commit 3b4f72e into vrm-c:master Oct 24, 2024
1 check passed
@ousttrue ousttrue deleted the fix/expression_override branch October 24, 2024 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants