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

MA Parametersで「内部値」とした値がMA Menu Installerの値と連携されない #305

Closed
33linn opened this issue May 17, 2023 · 5 comments · Fixed by #314
Closed

MA Parametersで「内部値」とした値がMA Menu Installerの値と連携されない #305

33linn opened this issue May 17, 2023 · 5 comments · Fixed by #314

Comments

@33linn
Copy link

33linn commented May 17, 2023

Issueの記載が初めてなので不備があればすみません。

◆バージョン
事象が発生しているのは1.5.0です(nadena.dev.modular-avatar-1.5.0.unitypackage)
1.4.5(nadena.dev.modular-avatar-1.4.5.unitypackage)では問題ありませんでした。
Unityは2019.4.31f1 VRCSDKは3.2.0です。

◆条件
MA Parametersで「内部値」にチェックを入れたboolがあり、
それをMA Menu Installerで導入するExMenuでToggle操作しようとしています。

・MA Parameters（buzzer_is_showが対象のパラメータ)
image

・MA Menu Installer（buzzer_is_showをToggleで操作)
image

以上の設定があるプレハブを投入し、再生ボタンを押して実行後
アバターのAvatar DescriptorからMenuを確認すると、1.5.0ではParameter名が内部値と同じ名前になっておらず、操作することができません。

・1.5.0の場合(Parameter名が内部値の名前にならず、操作できない)
image

・1.4.5の場合(Parameter名が内部値の名前になっていて連携されている)
image
@suzuryg
Copy link

suzuryg commented May 17, 2023

こちらは remapTo を使用していましたが、同様の現象が発生しています。

◆バージョン
MA 1.4.5で正常動作、MA1.5.0で発生
Unityは2019.4.31f1 VRCSDKは3.2.0

◆条件
MA ParametersでremapToを使用し、パラメータ名を変更
そのパラメータを使用するExpression Menuを Menu Installerで導入

◆結果
Expression Parameter のパラメータ名は変更されるが、Expression Menuのパラメータ名は変更されない
（RenameParametersHook.ProcessMenuInstaller で _context.PostProcessControls に登録した処理が実行されていない？）

@SakumofuRuki
Copy link

同じ現象です。一旦内部値のチェックを外して（名前被りがなければおそらく問題ないので）対処しています。

@Narazaka
Copy link
Contributor

アッ1.4までは動いてたんですね（1.5で仕様だと思って内部値チェック外した勢

@Narazaka
Copy link
Contributor

ちょうどこのバグを踏むアセットとかもある模様（内部値設定になっていた）

https://booth.pm/ja/items/4198627

@suzuryg suzuryg mentioned this issue May 22, 2023
Closed
bdunderscore added a commit that referenced this issue May 23, 2023
@bdunderscore
fix: internal parameters not renamed on installed VRCExpressionMenu s…
e7db3df 
…ubmenus

Thanks to suzuryg for finding the cause of the bug and suggesting an initial fix.

Reported-By: 33linn
Fixes: #305
@bdunderscore bdunderscore mentioned this issue May 23, 2023
Merged
bdunderscore added a commit that referenced this issue May 23, 2023
@bdunderscore
fix: internal parameters not renamed on installed VRCExpressionMenu s…
eb5edfd 
…ubmenus

Thanks to suzuryg for finding the cause of the bug and suggesting an initial fix.

Reported-By: 33linn
Fixes: #305
bdunderscore added a commit that referenced this issue May 23, 2023
@bdunderscore
fix: internal parameters not renamed on installed VRCExpressionMenu s…
a4c3e1d 
…ubmenus (#314)

Thanks to suzuryg for finding the cause of the bug and suggesting an initial fix.

Reported-By: 33linn
Fixes: #305
@Narazaka
Copy link
Contributor

こちらの問題まだ部分的に発生します。

以下のようにMenu ItemコンポーネントでSub Menuを参照した状態の場合にSub Menuのパラメーターが変更されないです。
image

親メニュー自体もアセットを参照して構成すると正常に動作します。
image

@Narazaka Narazaka mentioned this issue Jun 5, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@Narazaka @33linn @SakumofuRuki @suzuryg