Skip to content

Conversation

@rikunosuke
Copy link
Contributor

issue

  • [DM] データセットでライセンスを設定できる

仕様

  • 引数名は license で (Optional)

@rikunosuke rikunosuke requested a review from Jiei-S August 25, 2023 05:37
@rikunosuke rikunosuke self-assigned this Aug 25, 2023
@rikunosuke
Copy link
Contributor Author

@Jiei-S
PR作成したので、お手隙に確認をお願いいたします。

Copy link
Contributor

@Jiei-S Jiei-S left a comment

Choose a reason for hiding this comment

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

@rikunosuke
コメントしました

```python
dataset = client.create_dataset(
name="object-detection", # Only lowercase alphanumeric characters + hyphen is available
license="The MIT License" # Optional
Copy link
Contributor

Choose a reason for hiding this comment

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

[IMO]
どこかに固定のライセンス一覧を記載しても良さそうです。
(ユーザからすると、何のライセンスがあるか分からない)

※ 公開どドキュメントに記載する場合は不要かも

Copy link
Contributor

Choose a reason for hiding this comment

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

こちら返信ないですが、対応方針はどうなりますか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

公開ドキュメントで対応しようと思います。

@rikunosuke
Copy link
Contributor Author

@Jiei-S
修正をしました。

Copy link
Contributor

@Jiei-S Jiei-S left a comment

Choose a reason for hiding this comment

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

@rikunosuke
コメントしてます
(コード修正不要なら approve します)

```python
dataset = client.create_dataset(
name="object-detection", # Only lowercase alphanumeric characters + hyphen is available
license="The MIT License" # Optional
Copy link
Contributor

Choose a reason for hiding this comment

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

こちら返信ないですが、対応方針はどうなりますか?

@rikunosuke
Copy link
Contributor Author

@Jiei-S
こちら公開ドキュメントで対応しようと思います。

@Jiei-S
Copy link
Contributor

Jiei-S commented Aug 31, 2023

application 側に合わせてマージ

@Jiei-S Jiei-S merged commit 6eb8d51 into main Sep 1, 2023
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