Skip to content

Small improvements#3

Open
nagayev wants to merge 2 commits intowb-08:masterfrom
nagayev:master
Open

Small improvements#3
nagayev wants to merge 2 commits intowb-08:masterfrom
nagayev:master

Conversation

@nagayev
Copy link

@nagayev nagayev commented Oct 5, 2022

Add instruction how to install opencv
Fix part_4.py for modern opencv.

@nagayev nagayev mentioned this pull request Oct 5, 2022
Copy link
Owner

@wb-08 wb-08 left a comment

Choose a reason for hiding this comment

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

In readme file you can write pip install -r requirements.txt instead of pip install opencv-python

@nagayev
Copy link
Author

nagayev commented Oct 6, 2022

@wb-08 Yes, you are right, but there is only one requirement, so in my opinion the easiest way is to install opencv directly

@wb-08
Copy link
Owner

wb-08 commented Oct 6, 2022

@nagayev , when you install opencv directly, then it installs the newest version of the library. And there is a chance that something will change in the new version again. And you can also please write a more specific version of opencv using ==

@nagayev
Copy link
Author

nagayev commented Dec 22, 2022

@wb-08 Yes, you are right, but installing direct version of any heavy package like opencv makes your hard-drive unhappy.
I think the best way is installing opencv of certain range. E.g <5.0, because 5.0 is the next major release.

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