-
Notifications
You must be signed in to change notification settings - Fork 34
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
Add docstring to kernelkronmat
and small corrections
#377
Conversation
kernelkronmat
and small corrections
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.
Not sure I am around long enough to approve PRs, but it does look good to me.
@willtebbutt @devmotion Given that I restricted the function signatures and removed a meaningless kwarg, should that be considered breaking? |
My inclination is to say that this should not be considered breaking -- the restriction is probably a bug fix, and the meaningless kwarg feels like a bug. @devmotion what are your thoughts? |
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.
I'd consider it non-breaking.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: st-- <st--@users.noreply.github.com>
Summary
kernelkronmat
kernelkronmat
obsdim
kwarg@assert
statements byerror
iskroncompatible
docstringProposed changes
Following #369 I wanted to take care of moving
kernelpdmat
in the right place and add a docstring tokernelkronmat
What alternatives have you considered?
Deleting
kernelkronmat
Breaking changes
I added stronger requirements on the
kernelkronmat
which might be considered breaking...