New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(opendataset): update module docstring for opendataset #1115
docs(opendataset): update module docstring for opendataset #1115
Conversation
Pull Request Test Coverage Report for Build 1493734003
💛 - Coveralls |
@@ -4,6 +4,7 @@ | |||
# | |||
# pylint: disable=invalid-name | |||
# pylint: disable=missing-module-docstring | |||
"""Dataloader of AnimalPose5 dataset and AnimalPose7 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.
Dataloaders, other docstrings are also need to change
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.
whether there are more than two datasets I need to change to the dataloaders or I have to change all the dataloader to loaders
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.
use "Dataloaders" when there are two datasets
tensorbay/opendataset/AADB/loader.py
Outdated
@@ -3,6 +3,7 @@ | |||
# Copyright 2021 Graviti. Licensed under MIT License. | |||
# | |||
# pylint: disable=invalid-name, missing-module-docstring |
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 missing-module-docstring
comment can be removed
tensorbay/client/struct.py
Outdated
@@ -435,14 +435,17 @@ def _loads(self, contents: Dict[str, Any]) -> None: | |||
self.committer = User.loads(contents["committer"]) | |||
|
|||
|
|||
class Draft(AttrsMixin, ReprMixin): | |||
class Draft(AttrsMixin, ReprMixin): # pylint: disable=too-many-instance-attributes |
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.
your PR includes commits from other friends, it need to be fixed.
9cfc5dd
to
1cb9e68
Compare
@@ -2,7 +2,8 @@ | |||
# |
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.
commit message is too long
update module docstring for opendataset
is enough
7c8c56f
to
f25d642
Compare
@@ -1,8 +1,9 @@ | |||
#!/usr/bin/env python3 | |||
# | |||
# Copyright 2021 Graviti. Licensed under MIT License. | |||
# | |||
# pylint: disable=invalid-name, missing-module-docstring | |||
|
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.
do not need to delete #
f25d642
to
53ed9f3
Compare
53ed9f3
to
8ecf071
Compare
No description provided.