Skip to content
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

task done #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

task done #1

wants to merge 2 commits into from

Conversation

ajit2704
Copy link

No description provided.

@athityakumar
Copy link
Member

Hey @ajit2704 , this is definitely a good attempt 👍. But, there are some issues / suggestions that I have described below - do have a look at them.

  • I get this error while running the auv.py script. You aren't getting any error? Which Python and OpenCV version(s) have you tested on?
    image

  • Move the script from graph-detect/auv.py to graph-detect/<username>/auv.py. All your files should be put up inside graph-detect/<your_username>/ directory.

  • Use more informative commit and PR messages. These messages should reflect on the changes you have done - which task done commit doesn't accomplish. No problem, but take care of this in future commits & PRs.

@ajit2704
Copy link
Author

ajit2704 commented Feb 26, 2017 via email

@athityakumar
Copy link
Member

@ajit2704 - Oh, okay. But I'm still getting the error. Does the auv.py provide proper output on your laptop? Anyway, make the changes and push to your fork's master branch. Changes will automatically appear in the Pull Request.

@ajit2704
Copy link
Author

ajit2704 commented Feb 26, 2017 via email




image,contours,hierarchy = cv2.findContours(thresh.copy(),cv2.RETR_LIST,cv2.CHAIN_APPROX_SIMPLE)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: need more than 2 values to unpack

Have a look at documentation of the findContours() function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants