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

#8426 SDKで自動アノテーションを実行できる #236

Merged
merged 5 commits into from
Dec 10, 2024

Conversation

kmkkiii
Copy link
Collaborator

@kmkkiii kmkkiii commented Dec 5, 2024

サマリ

概要、背景

Issue:https://github.com/fastlabel/fastlabel-application/issues/8426
API・Batchの変更PR:https://github.com/fastlabel/fastlabel-application/pull/8516

(不具合の場合のみ) 発生原因

対応内容

やったこと

  • SDKクライアントへ自動アノテーションを実行するメソッドを追加
  • README追記
  • example追加

やれていないこと、妥協点

テスト

変更の意図に沿った基本動作が確認できている

  • SDKから自動アノテーションが実行できること
  • 読み取り専用のAPIアクセストークンを使用している場合、SDKから自動アノテーションが実行できないこと
  • バリデーション
    • 自動アノテーションが利用できないプロジェクトを指定した場合は実行できないこと
    • 指定したモデル名のモデルが存在しない場合は実行できないこと
    • アノテーション紐付けが必要なモデルを指定して、マッピング未完了な場合は実行できないこと
    • 既存のアノテーションを上書き(update_existing)オプションが使用できないプロジェクト・モデルでupdate_existing=trueを指定した場合実行できないこと
  • SDKで実行した自動アノテーションで追加されたラベルが利用状況に計上されていること
  • ラベル数上限を超えている場合、SDKから自動アノテーションが実行できないこと
  • SDKによる各プロジェクト・モデルの自動アノテーション実行
    • APIキーが必要なモデルについてはステージング環境で確認

関連する既存機能にデグレがないことを確認

  • 各プロジェクト・モデルの自動アノテーション実行
    • APIキーが必要なモデルについてはステージング環境で確認

関連リンク

補足

@kmkkiii kmkkiii self-assigned this Dec 5, 2024
Copy link
Contributor

@ryoKaz ryoKaz left a comment

Choose a reason for hiding this comment

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

LGTM

@ryoKaz ryoKaz merged commit 70b0fb6 into main Dec 10, 2024
1 check passed
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