Skip to content

Conversation

@ijufumi
Copy link
Contributor

@ijufumi ijufumi commented Dec 4, 2021

@ijufumi ijufumi requested a review from faycute December 4, 2021 01:22
@ijufumi ijufumi self-assigned this Dec 4, 2021
external_status: str = None,
tags: list = [],
annotations: list[dict] = [],
annotations: List[dict] = [],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

3.9から入った文法が使われていて、手元で動かなかったので修正しています。
https://www.python.org/dev/peps/pep-0585/

Copy link
Contributor

Choose a reason for hiding this comment

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

ありがとうございます!

if len(contents) == 250:
raise FastLabelInvalidException(
"The count of files should be under 250", 422)

Copy link
Contributor

Choose a reason for hiding this comment

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

こちらですが、Readmeのこの辺りに、複数画像プロジェクトの1タスクのファイル数上限について記載いただけますでしょうか?
https://github.com/fastlabel/fastlabel-python-sdk#create-task-2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ryoKaz
ありがとうございます!
記載してみました。
6078277

@ryoKaz ryoKaz merged commit 1309785 into develop Dec 9, 2021
@ryoKaz ryoKaz deleted the feature/973-limitation-images branch December 9, 2021 13:13
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