Getting python wrappers to work, assertions (Feature #1005)


Added by Martin Bergtholdt almost 14 years ago. Updated over 13 years ago.


Status:Done Start date:
Priority:High Due date:
Assignee:- % Done:

0%

Category:python bindings
Target version:-
Difficulty: Pull request:

Description

I have spent quite some time to get the Python wrappers to work, specifically the OpenCV 2.x API.

My platform is Windows-7, Visual Studio 2008 64bit, Python 2.7 64bit. Everything seems to work now, I did have to apply these patches, though. Could someone have a look please and integrate them?

There is still a desirable extension when doing interactive scripting in python:

The CV_Assert macro causes the complete Python interpreter to stop when certain arguments are not in the proper form (e.g. a numpy array is not contiguous). Creating a runtime exception would certainly help here since an interactive python session would simply be able to yield an error without crashing the complete python interpreter.


patch.zip (3.4 kB) Martin Bergtholdt, 2011-04-14 05:20 pm


Associated revisions

Revision 23d211bf
Added by Vadim Pisarevsky over 13 years ago

applied patches from #1005

Revision 9cbeea03
Added by Roman Donchenko over 11 years ago

Merge pull request #1005 from alekcac:doc_fix

Revision 13cd0a05
Added by Roman Donchenko over 11 years ago

Merge remote-tracking branch 'origin/2.4'

Original pull requests:
#996 from jet47:gpu-nvcuvid-libraries
#995 from jet47:fix-bug-2985
#999 from snosov1:unreliable-results-fix
#1005 from alekcac:doc_fix
#1004 from jet47:fix-bug-3068
#987 from jet47:bug-3085-fix
#969 from pengx17:2.4_binary_cache
#929 from dominikrose:mingw-libdc1394-2-windows
#1000 from ivan-korolev:fix_sift_bug_2892
#1001 from ivan-korolev:fix_stitching_bug_2405
#998 from asmorkalov:android_cmake_mips_fix
#993 from ivan-korolev:fix_videostab_bug_3023
#988 from snosov1:3071-fix
#986 from pengx17:2.4_initiated_context
#982 from pengx17:2.4_fix_two_bugs
#981 from SeninAndrew:ximea_camera_support_fix
#991 from asmorkalov:android_javadoc_fix
#972 from jet47:mog2-params-bug-2168
#980 from SpecLad:include-config
#973 from pengx17:2.4_oclclahe
#903 from aks2:2.4
#968 from asmorkalov:android_na_cproj_fix
#971 from SpecLad:matchers-ctor
#970 from asmorkalov:dshow_valid_check_fix
#965 from apavlenko:fix_java_empty_mats

Conflicts:
cmake/OpenCVModule.cmake
modules/core/src/matmul.cpp
modules/gpu/CMakeLists.txt
modules/ocl/include/opencv2/ocl/ocl.hpp
modules/ocl/perf/perf_imgproc.cpp
modules/ocl/src/imgproc.cpp
modules/ocl/src/initialization.cpp
modules/stitching/src/matchers.cpp
modules/video/src/video_init.cpp
modules/videostab/src/global_motion.cpp

History

Updated by Vadim Pisarevsky over 13 years ago

thank you!

the patches 0001 and part of 0003 have been applied in r5460.
changes in the rest of the patch 0003 and 0004 have been made earlier.

patch 0002 was not applied. In the case of matrix and the missing output parameter pyopencv_from() still initializes a matrix with a numpy allocator. That saves us an extra matrix copy operation in pyopencv_from(). If the current solution does not work, a test sample would be welcome.

  • Status changed from Open to Done
  • (deleted custom field) set to fixed

Also available in: Atom PDF