Repository navigation
Conversation
音量は、その tick に描画された看板のうち最も近いものまでの距離で決めている。 看板に背を向けると、看板は視界の外になって描画されなくなり、距離が分からない tick ができる。 そのときに距離を 0 (目の前) と見なしていたため、再生が止まるまでの約 3 秒間、最大の音量になっていた。 - 描画されなかった tick は、最後に分かった距離のまま音量を決める - 動作確認 (SelfTest): 動画・音声の看板では、正面を向いたときと、真上を向いて看板が視界から外れたときの 音の大きさを比べ、外れたときの方が大きければ失敗にする
Member
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #19
原因
動画・音声の音量は、その tick に描画された看板のうち、最も近いものまでの距離で決めています (
core/.../sign/MediaSession.java)。看板に背を向けると、看板は視界の外になって描画されなくなり、その tick は距離が分かりません。
このとき距離を 0 (目の前) と見なしていたため、再生が止まるまでの約 3 秒間、最大の音量で鳴っていました。
各バージョンの実装 (
adapters/) が渡しているカメラから看板までの距離は正しく、問題は全バージョン共通の core 側でした。変更
SelfTest): 動画・音声の看板では、正面を向いたときと、真上を向いて看板が視界から外れたときの音の大きさを比べ、外れたときの方が大きければ失敗にします。結果の文に「facing / looking away の音の大きさ」と「外れている間に描画された回数」を書きます (視界から外れても描画が続くバージョンでは、比較の意味が薄いことが分かるように)。
確認
./gradlew :dist:assemble: 成功。コンパイラーの警告は main と同じ (この変更で増えたものはなし)tools/verify): 下に追記します