-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Added doc strings to wandb logger #9109
Conversation
for more information, see https://pre-commit.ci
Does this need to be more verbose, because I think it should but I don't know how. |
Codecov Report
@@ Coverage Diff @@
## master #9109 +/- ##
=======================================
- Coverage 92% 88% -4%
=======================================
Files 178 176 -2
Lines 14694 14666 -28
=======================================
- Hits 13526 12889 -637
- Misses 1168 1777 +609 |
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 !
What does this PR do?
Links #9028
Adds docstrings in the wandb logger class.
Does your PR introduce any breaking changes? If yes, please list them.
None
Before submitting
PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:
Did you have fun?
Make sure you had fun coding 🙃