Skip to content

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