Skip to content

Conversation

@ammkk
Copy link

@ammkk ammkk commented Jul 22, 2024

背景

create_dataset_objectで分類アノテーションを付与する際のパラメータについてREADMEに記載が無かった。

対応

分類アノテーションを付与する時のパラメータについてREADMEに記載した。

@ammkk ammkk self-assigned this Jul 22, 2024
],
"attributes": [
{
"type": "text",
Copy link
Author

Choose a reason for hiding this comment

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

必須のパラメータであり、現状のサンプルコードではエラーが出てしまう。

README.md Outdated
}
```

If you would like to create with classification type annotations, please pass empty points and value of the annotation named 'classification'.

Choose a reason for hiding this comment

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

create の後に目的語(a new dataset objectとか)あった方が良さそうです!
If you would like to create a new dataset object with classification type annotations, ~~

README.md Outdated
Comment on lines 2593 to 2620
```

If you would like to create with classification type annotations, please pass empty points and value of the annotation named 'classification'.

```python
dataset_object = client.create_dataset_object(
dataset="YOUR_DATASET_NAME",
name="brushwood_dog.jpg",
file_path="./brushwood_dog.jpg",
tags=["dog"], # max 5 tags per dataset object.
licenses=["MIT", "my-license"], # max 10 licenses per dataset object
annotations=[
{
"type": "classification",
"value": "classification",
"points": [],
"attributes": [
{
"type": "text",
"value": "Scottish field",
"key": "kind"
}
]
}
]
)
```

Choose a reason for hiding this comment

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

Response Dataset Objectのセクションより前にあった方が良さそうかと思いましたがいかがでしょうか?

@takahiro-tamenishi takahiro-tamenishi merged commit b972dc6 into main Jul 23, 2024
@takahiro-tamenishi takahiro-tamenishi deleted the feature/update-dataset-readme branch July 23, 2024 01:28
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.

4 participants