search

hecomi / uLipSync

MFCC-based LipSync plug-in for Unity using Job System and Burst Compiler
https://tips.hecomi.com/entry/2022/01/30/152519
MIT License
1.07k stars 128 forks source link

The first parameter set in uLipSyncAnimator does not work. #37

Closed Muta-K closed 1 year ago

Muta-K commented 1 year ago

uLipSyncAnimatorで一番最初にセットしたパラメータが動作しないようです。

原因を調査すると、uLipSyncAnimatorEditorでセットするuLipSyncAnimator.AnimatorInfo.indexが-1になっており、 uLipSyncAnimator.OnApplyAnimator内のif文で弾かれてしまうようです。

解決策としては、uLipSyncAnimatorEditor.DrawParameterListItem内の157行目から始まるindex関連の処理で、 ±１している操作を修正すれば解決できますが、 これをしてしまうと、既存のユーザーがアップデート時に不具合が出てしまう恐れがありそうです。

何か解決方法、もしくは修正をお願いできないでしょうか？

image

hecomi commented 1 year ago

ご報告ありがとうございます。バグっていますね。。

次回リリースで Profile の仕様変更に関係してメジャーバージョンを上げる予定をしておりまして、そのときに併せて破壊的変更として修正いたします。バグ自体はリリースに先立ち、main ブランチで更新を行います。

hecomi commented 1 year ago

こちらの修正を v3.0.0 に組み込みました： https://github.com/hecomi/uLipSync/releases/tag/v3.0.0