-
Notifications
You must be signed in to change notification settings - Fork 303
Conversation
ce42b1a
to
79cc343
Compare
79cc343
to
f7eb7f3
Compare
docs/source/reference/utils.rst
Outdated
@@ -79,6 +75,10 @@ assert_is_image | |||
~~~~~~~~~~~~~~~ | |||
.. autofunction:: assert_is_image | |||
|
|||
assert_is_label_dataset | |||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The length of ~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
docs/source/reference/datasets.rst
Outdated
@@ -3,9 +3,9 @@ Datasets | |||
|
|||
.. module:: chainercv.datasets | |||
|
|||
DirectoryParsingClassificationDataset | |||
DirectoryParsingLabelDataset | |||
------------------------------------- |
There was a problem hiding this comment.
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 -
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh. Sorry...
@@ -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 |
There was a problem hiding this comment.
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
?
@Hakuyume |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Follows discussion in #399.
ClassificationDataset has been added after updating the version to
0.6.0
.Therefore, this PR is not
no-compat
.