-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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] fix docstring #1578
[Docs] fix docstring #1578
Conversation
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.
It is exactly a typo. Thanks for your contribution.
Codecov ReportBase: 87.90% // Head: 87.90% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## dev-1.x #1578 +/- ##
========================================
Coverage 87.90% 87.90%
========================================
Files 399 399
Lines 26015 26015
Branches 4044 4044
========================================
Hits 22868 22868
Misses 2240 2240
Partials 907 907
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@dienachtderwelt BTW, if you are interested in image editing algorithms, we are looking forward to more contribution. Lets make this repo better together. 😃 |
Dear dienachtderwelt, |
Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
First PR. Just a mini doc fix as a try LOL.
Modification
Change argname in docstring to match the actual arguments.
Checklist
Before PR:
After PR: