I changed the ug_highgui to the ug_kinect. However, I am expecting this to create 2 files. How can I tell in a pull request that the original ug_highgui can be deleted? Someone?

@StevenPuttemans, may be git mv will be good for this?

I will make the changes as suggest. As to the mv, since I renamed the original file, I have no idea how I have to do a mv on my system now ... ill try with rm or something like that. Get back to you!

It seems a rename and then followed by a remove of the old file did the trick. Lets merge commits now!

It should be correct now. Saw that I also removed a subtitle to much, fixed that now.

Hi Steven! Thanks for PR!

@StevenPuttemans Do you have any thoughts, why two builders were crushed?

Afaik this has nothing to do with this pull request. Others have same error, seems the bug is in the python bindinh of a Mat function.

We just need to wait until someone fixes this I guess

👍

Also it seems that this problem only lives in the 2.4 branch. Is there a way of checking how this problem was fixed in master branch?

I think it is kind of weird that changing docs results in this error

test_boundscatch (main.AreaTests) ... OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtr2D, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1754
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtr2D, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1754
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893
OpenCV Error: One of arguments' values is out of range (index is out of range) in cvPtrND, file C:\builds_ocv\precommit_windows64\opencv\modules\core\src\array.cpp, line 1893

This makes me believe that there is still a problem with current 2.4 branch in the python tests.

@Daniil-Osokin Also to add to your remark. The identations were already there, I guess they are accepted because they are inside comment brackets? Would you like me to change them?

Ok it seems this was fixed in #2914
I am going to update the fix onto the latest 2.4 release! Will keep in touch!

I tried fixing it, lets see if I solved it!

Steven,
can you please redo the PR? if you check the difference, you will see that it modifies several other files that it should not have done. I actually suggest to create the new PR from scratch (take the modified .rst files)

I will redo it :) next time lets not wait weeks to add little changes like this :D Prolly I did somethng wrong or new changes are interfering :D

#2985 here you go :)