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

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

Closed
Muta-K opened this issue Apr 14, 2023 · 2 comments
Closed

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

Muta-K opened this issue Apr 14, 2023 · 2 comments

Comments

@Muta-K
Copy link

Muta-K commented Apr 14, 2023

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

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

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

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

image

@hecomi
Copy link
Owner

hecomi commented Apr 16, 2023

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

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

hecomi added a commit that referenced this issue Apr 17, 2023
@hecomi
Copy link
Owner

hecomi commented Apr 27, 2023

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

@Muta-K Muta-K closed this as completed May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants