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

長尺の音声をベイクすると"Hold on"でハングする問題を修正 #56

Merged
merged 1 commit into from
Dec 29, 2023

Conversation

mkc1370
Copy link
Contributor

@mkc1370 mkc1370 commented Oct 10, 2023

概要

長尺(10分以上?)の音声のBakedDataをベイクすると"Hold on"が表示されたままになる問題を修正しました。

変更内容

  • Baked DataのDataに表示されるグラフのPhonemeごとのフレーム数が512個を超えないように処理を変更
    恐らく、このPRで書いた処理内容が本来意図されていたものかと思います。

詳細

Handles.DrawAAPolyLine()に渡すポイントの数が大きくなると"Hold on"が表示されたままになるようでした。
具体的なポイントの数は状況によって前後するため不明ですが、大体80kを超えるとハングしやすくなります。
(Handles.DrawAAPolyLineの挙動に関してはUnityにバグレポートを送信済みです)

image

@hecomi hecomi merged commit 91658e9 into hecomi:main Dec 29, 2023
@hecomi
Copy link
Owner

hecomi commented Dec 29, 2023

ご修正ありがとうございます。
マージが遅れてしまい大変申し訳ありませんでした…!

@mkc1370
Copy link
Contributor Author

mkc1370 commented Dec 30, 2023

ご確認ありがとうございます!

Handles.DrawAAPolyLineの挙動については、バグではないとのことでした
今回のPRのように、Handleに渡す個数を制限する対応するのが無難そうです
Unity Issue Tracker - Editor freezes when inserting a bug number to the array of points in “Handles.DrawAAPolyLine”

@mkc1370 mkc1370 deleted the fix/baked_data_preview branch December 30, 2023 07:42
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

Successfully merging this pull request may close these issues.

2 participants