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
AddLinearOperator [Linear Operator Helper Class] #3679
Conversation
|
||
# repr helper functions | ||
def indent(s, nspace): | ||
"""give indentation of the second line and next lines |
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.
Please add a note this function is used to generate string representations of LinearOperator (and any other classes it is used for)
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.
@GreatRSingh Please rebase this PR on top of main and address open comments
a6f48ec
to
baf3370
Compare
""" | ||
|
||
|
||
def indent(s, nspace): |
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.
In a future PR, please add unit tests for these utilities
"""Convert the shape to string representation. | ||
It also nicely formats the shape to be readable. | ||
# Helper Classes | ||
class AddLinearOperator(LinearOperator): |
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 should be added to the documentation. Can be done in future PR
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
Description
Fix #(issue)
AddLinearOperator [Linear Operator Helper Class]
Type of change
Please check the option that is related to your PR.
Checklist
yapf -i <modified file>
and check no errors (yapf version must be 0.32.0)mypy -p deepchem
and check no errorsflake8 <modified file> --count
and check no errorspython -m doctest <modified file>
and check no errors