Skip to content

Conversation

@ammkk
Copy link

@ammkk ammkk commented Sep 22, 2023

概要

  • 動画タスク作成時に動画のコーデックがH.264でない場合はエラーを返すようにしました

背景

  • SDKでの動画タスク作成の場合、MP4ファイルのH.264でない場合はFastLabelのアプリで動画を正常に表示することができません
  • 拡張子がmp4であるかどうかのバリデーションは行われていましたが、拡張子がmp4であってもコーデックがHEVCなどサポートしていなコーデックの場合がありました
  • そのため、コーデックについてのバリデーションを追加しました

やったこと

  • コーデックのバリデーションの追加

動作確認

スクリーンショット 2023-09-21 17 48 30 スクリーンショット 2023-09-21 17 50 23

@ammkk ammkk self-assigned this Sep 22, 2023
@ammkk ammkk changed the title SDKで動画のタスク登録するときに、動画のコーデックがMP4のものではない場合はエラーを返す SDKで動画のタスク登録するときに、動画のコーデックがH.264のものではない場合はエラーを返す Sep 22, 2023
)
if not utils.is_video_supported_codec(file_path):
raise FastLabelInvalidException(
"Supported codec is H.264.", 422
Copy link
Contributor

@takahiro-tamenishi takahiro-tamenishi Sep 22, 2023

Choose a reason for hiding this comment

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

[imo]
SDKを用いた登録では、H.264のエンコードのみ登録が可能です。という意味合いが良いと思います!
現状のメッセージだと、画面からのインポートもできないようにユーザーは感じそうです。

あとSDK側のReadmeにも記載をしたいです!

Copy link
Author

Choose a reason for hiding this comment

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

エラーメッセージの修正とReadmeの更新を行いました!

@takahiro-tamenishi takahiro-tamenishi merged commit b19eabb into main Sep 22, 2023
@takahiro-tamenishi takahiro-tamenishi deleted the feature/codec-restriction-h264 branch September 22, 2023 06:27
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.

3 participants