sightmachine/SimpleCV

README.md: bad PATH and PYTHONPATH setting for windows

fasterpython opened this issue · 0 comments

In README.md, it says:
SETX PATH C:/Python27/;C:/Python27/Scripts/;C:/OpenCV2.3/opencv/build/x86/vc10/bin/;%PATH%
SETX PYTHONPATH C:/OpenCV2.3/opencv/build/python/2.7/;%PYTHONPATH%

however, the correct one should be:
SETX PYTHONPATH C:\OpenCV2.3\opencvbuild\python\2.7;%PYTHONPATH%
SETX PATH C:\Python27;C:\Python27\Scripts;C:\OpenCV2.3\opencv\build\x86\vc10\bin;%PATH%

and also, it's only correctly in 32-bit python (x86 or x64 cpu, but 32-bit python)
if the platform is x64 cpu and 64-bit python, it should be:
SETX PATH C:\Python27;C:\Python27\Scripts;C:\OpenCV2.3\opencv\build\x64\vc10\bin;%PATH%

==========================================================

and also, since the latest opencv2.4.13.6 (released on 2018-2), the correct PYTHONPAHT should be:
64bit python:
SETX PYTHONPATH C:\OpenCV2.3\opencvbuild\python\2.7\x64;%PYTHONPATH%
or for 32bit python:
SETX PYTHONPATH C:\OpenCV2.3\opencvbuild\python\2.7\x86;%PYTHONPATH%

Because OpenCV2.3.1 was released on 2011, 7 years ago, that's a too too old version for users now.
when I follow README.md

Download OpenCV 2.3 Superpack: http://sourceforge.net/projects/opencvlibrary/files/opencv-win/2.3.1/OpenCV-2.3.1-win-superpack.exe/download

there is a error that:
Cannot load OpenCV library which is required by SimpleCV

so I download the latest opencv2.4.13.6 (released on 2018-2) and setup correct paths, and it all works fine now.

Please update the README.md as suggest before, thanks!