Skip to content

Added OpenCV to Image Processing#635

Closed
shivenmian wants to merge 6 commits into
realpython:masterfrom
shivenmian:master
Closed

Added OpenCV to Image Processing#635
shivenmian wants to merge 6 commits into
realpython:masterfrom
shivenmian:master

Conversation

@shivenmian
Copy link
Copy Markdown
Contributor

Along with PIL, one very important library namely OpenCV is also widely used in Image Processing. Wrote an introduction to this too. Documentation link added to it.

@sigmavirus24
Copy link
Copy Markdown
Contributor

(Pinging travis to run against this)

@shivenmian
Copy link
Copy Markdown
Contributor Author

Fixed target name. All checks have passed. :)

Comment thread docs/scenarios/imaging.rst Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • or OpenCV in short -> more commonly known as OpenCV
  • is a more advanced image manipulation ...

(it is more useful depending entirely on what you have to do)

@sigmavirus24
Copy link
Copy Markdown
Contributor

@shivenmian when you've updated the document with the changes suggested in-line, let me know.

@shivenmian
Copy link
Copy Markdown
Contributor Author

I don't know where the NumPy reference is. I checked this but it only has an Intro to NumPy and it does not have the installation procedure but just a direct link to the official site.

Rest changes have been made. Please have a look, and let me know if something additional needs to be done. :)

Comment thread docs/scenarios/imaging.rst Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

... in this `collection of tutorials <...>`_

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

@shivenmian
Copy link
Copy Markdown
Contributor Author

I've updated the title underlines too. Please tell me if there are any more changes left.

@sigmavirus24
Copy link
Copy Markdown
Contributor

I don't think so. I'm going to rebase this before I merge it into master to squash /edit some of your commits/commit messages. This will be closed by a merge commit. Thanks @shivenmian

@shivenmian
Copy link
Copy Markdown
Contributor Author

Thank you! :D

@sigmavirus24
Copy link
Copy Markdown
Contributor

🍰 @shivenmian

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