Skip to content
This repository has been archived by the owner on Jul 2, 2021. It is now read-only.

Use LabelDataset naming convention #405

Merged
merged 10 commits into from
Sep 28, 2017
Merged

Conversation

yuyu2172
Copy link
Member

@yuyu2172 yuyu2172 commented Aug 21, 2017

Follows discussion in #399.

ClassificationDataset has been added after updating the version to 0.6.0.
Therefore, this PR is not no-compat.

@yuyu2172 yuyu2172 added this to the v0.7 milestone Aug 22, 2017
@@ -79,6 +75,10 @@ assert_is_image
~~~~~~~~~~~~~~~
.. autofunction:: assert_is_image

assert_is_label_dataset
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Copy link
Member

Choose a reason for hiding this comment

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

The length of ~

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you!

@@ -3,9 +3,9 @@ Datasets

.. module:: chainercv.datasets

DirectoryParsingClassificationDataset
DirectoryParsingLabelDataset
-------------------------------------
Copy link
Member

Choose a reason for hiding this comment

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

Sorry. This line should also be fixed. (the length of -)

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh. Sorry...

yuyu2172 added a commit to yuyu2172/chainercv that referenced this pull request Sep 20, 2017
@@ -91,8 +91,8 @@ class DirectoryParsingClassificationDataset(chainer.dataset.DatasetMixin):
--- class_1
|-- img_0.png

>>> from chainercv.dataset import DirectoryParsingClassificationDataset
>>> dataset = DirectoryParsingClassificationDataset('root')
>>> from chainercv.dataset import DirectoryParsingLabelDataset
Copy link
Member

Choose a reason for hiding this comment

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

from chainercv.dataset -> from chainercv.datasets?

@yuyu2172
Copy link
Member Author

@Hakuyume
Can you merge?

Copy link
Member

@Hakuyume Hakuyume left a comment

Choose a reason for hiding this comment

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

LGTM

@Hakuyume Hakuyume merged commit 83244e7 into chainer:master Sep 28, 2017
@yuyu2172 yuyu2172 deleted the label-dataset branch September 28, 2017 00:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants