-
Notifications
You must be signed in to change notification settings - Fork 3
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
Foxiy/class #9
Foxiy/class #9
Conversation
* Added more comments to functions. Signed-off-by: Ondrej Jariabka <o.jariabka@gmail.com>
* Moved detection of a board to class Signed-off-by: Ondrej Jariabka <o.jariabka@gmail.com>
def get_board(self, image): | ||
""" | ||
Makes a copy of input image, applies a median blur to cartoon-ify the | ||
image. It gets rid of noise and not wanted colors. Calls function which will |
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 undestand this is probably just a tab but visually this is already further than 80 characters. Can we do something about it?
* Added function that takes as parameter normal image and returns conture of that image. Signed-off-by: Ondrej Jariabka <o.jariabka@gmail.com>
self.H, self.S, self.V]))) | ||
|
||
|
||
def timeit(func): |
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 believe this can be put out of the classes scope.
Signed-off-by: Ondrej Jariabka <o.jariabka@gmail.com>
46 22 2 83 181 137 |
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.
As I've mentioned earlier 36
worked better for me.
@FOXIY looks good to me |
No description provided.