Hotfix/2025.3.3 - #328
Merged
Merged
Conversation
- 3 秒 / 15 fps の H.264 MP4 を testdata に追加する - openh264 / VideoToolbox / nvcodec の各 engine を明示指定するテストを追加する
- キーフレーム到来時に SPS / PPS の変化を検知して内部の Decoder を再生成する - SPS / PPS 抽出関数を video_h264 / video_h265 に集約する
- hvc1 の Unknown box を hev1 として再パースするヘルパーを video_h265 に追加する - reader_mp4 で SampleEntry::Unknown/hvc1 を検知して SampleEntry::Hev1 に変換する
- 3 秒 / 15 fps / hvc1 の H.265 MP4 を testdata に追加する - VideoToolbox / nvcodec の各 engine を明示指定するテストを追加する
- finish() + handle_decoded_frames() で非同期処理中のフレームを回収してから作り直す - output_queue の空チェックは不要なので削除する
- 3 秒 / 15 fps の VP8 / VP9 / AV1 MP4 を testdata に追加する - libvpx / dav1d / nvcodec の各 engine を明示指定するテストを追加する - nvcodec 版は sample_entry ベースの検知では発火しないため #[ignore] にする
- shiguredo_nvcodec 2025.2.1 では H.265 の再作成直後に cuvidCreateDecoder が失敗する - 上流の対応が入ったら ignore を外す前提
- crates.io の =2026.2.0 に切り替えて hisui/crates/shiguredo_nvcodec/ を削除する - Decoder / Encoder が callback ベース API になったため NvcodecDecoder / NvcodecEncoder を書き換える - ストリーム中の解像度変化対応が上流に取り込まれたため hisui 側の reinitialize_if_need は撤去する
- shiguredo_nvcodec 2026.2.0 で pfnSequenceCallback の解像度変化対応が入ったため
- 案 A の実装で decoder_nvcodec からも使うために video_h264 / video_h265 に pub 移設していた - shiguredo_nvcodec 2026.2.0 への切替で decoder_nvcodec の reinit ロジックが不要になったため - decoder_video_toolbox.rs 内 private の元の形に戻す
- shiguredo_nvcodec 2026.2.0 の pfnSequenceCallback → destroy+create でも解像度変化を扱えなかった - H.264 / VP8 は pass しているのでこの 3 コーデックのみ暫定 ignore
- develop 側でも valgrind チェックを行っていないため - flaky なノイズを減らすことが目的
- 従来は初回のみ抽出して以降キャッシュし続けていた - 解像度変化などで sample_entry が更新された場合に古い parameter set を prepend し続ける
- 160x120 は NVDEC の HEVC/VP9/AV1 の最小デコード解像度 (144x144 等) を下回っていた - 全コーデック共通で解像度シーケンスを 320x240 -> 224x160 -> 320x240 に変える - H.265 / VP9 / AV1 の nvcodec テストの #[ignore] を外して CI で通ることを狙う
- CHANGES に 2026.2.0 UPDATE を追加し FIX 説明を実装に合わせる - 2026.2.0 API 追随の履歴コメントを削除・現状説明に書き換える
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
1 participant
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.
No description provided.