Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

既知の問題に OpenH264 不具合で合成が失敗する件を記載 #159

Merged
merged 7 commits into from
Sep 18, 2024

Conversation

miosakuma
Copy link
Contributor

OpenH264 の不具合で合成が失敗する事象について既知の問題を作成して記載しました。
一部、表記の修正をこの修正とは別に行なっています、該当箇所にコメントをします。

確認して欲しい事項は以下です

  • 発生した環境についての記載をどこまで書くか
    • 他の環境(例えば iOS Chrom)でも発生する可能性があります
    • タイトルは環境を記載せず広めに書いて、事象を確認しましたという書き方にしています
  • 回避策の有無の記載について
    • 「回避策はなく、OpenH264 の修正を待つ必要がある」などの記載をするべきか
    • あまり重要な情報ではないため記載は不要と判断しました

This pull request includes updates to the documentation to provide more comprehensive information on known issues and improve readability.

Documentation Updates:

  • README.md: Added a new section for known issues and included a link to the KNOWN_ISSUES.md file. Improved readability by adding a missing period.
  • doc/KNOWN_ISSUES.md: Created a new document to detail known issues, specifically addressing a problem with H.264 video synthesis failures on macOS and iOS Safari due to OpenH264 decoding issues.

@@ -60,12 +60,16 @@ Hisui を使ってみたい人は [USE.md](doc/USE.md) をお読みください

## ビルドする

Linux 版 Hisui をビルドしたい人は [BUILD_LINUX.md](doc/BUILD_LINUX.md) をお読みください
Linux 版 Hisui をビルドしたい人は [BUILD_LINUX.md](doc/BUILD_LINUX.md) をお読みください
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

他に合わせて末尾に「。」を追加しました。

@miosakuma miosakuma requested a review from torikizi September 18, 2024 01:53
Copy link
Contributor

@torikizi torikizi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

対応ありがとうございます。一点だけ確認をお願いします。あとは大丈夫です。

記載内容も現状で良いと思います。


## H.264 の映像について合成が失敗することがある

OpenH264(バージョン 2.4.1) のデコード不具合により macOS または iOS の Safari で録画した H.264 の映像を合成した時に失敗する事象を確認しています。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

バージョンまでは書かなくても良いかもしれません。
最新の 2.4.1 だけで出る問題ではなく、過去バージョンを使用しても回避する方法が(おそらく)ないので。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

了解しました。こちらで確認したバージョンがわかるように、という意図でした。
バージョン 2.4.1 で確認をしたことがわかるように文章を直すことも考えましたが、今後も OpenH264 のリリース時に不具合の確認はすると思いますので、削除するようにします。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6e63268 で対応しました

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!

@miosakuma
Copy link
Contributor Author

確認ありがとうございました、マージします

@miosakuma miosakuma merged commit 16913db into develop Sep 18, 2024
1 check passed
@miosakuma miosakuma deleted the feature/add-known-issue branch September 18, 2024 02:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants