-
Notifications
You must be signed in to change notification settings - Fork 360
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
DOC Fix various documentation formatting inconsistencies #1209
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1209 +/- ##
=======================================
Coverage 66.94% 66.94%
=======================================
Files 91 91
Lines 4359 4359
=======================================
Hits 2918 2918
Misses 1441 1441
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
|
||
"""Defines the ErrorAnalysisManager for discovering errors in a model. | ||
|
||
:param model: The model to analyze errors on. |
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.
This block is duplicated in the constructor, I assume?
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.
Description
We had several inconsistently formatted files, so this PR addresses a bunch of them. The idea is to have the class docstring explain the purpose of the class and the constructor docstring provide constructor args. So far, we occasionally had constructor args in both which leads to duplication and inevitably things get out of sync. I've fixed such an instance here as well.
There are also a few miscellaneous fixes in comments.
Areas changed
npm packages changed:
Python packages changed:
Tests
Screenshots (if appropriate):
Documentation: