Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reg: broken python sample #1935

Open
berak opened this issue Dec 7, 2018 · 4 comments
Open

reg: broken python sample #1935

berak opened this issue Dec 7, 2018 · 4 comments

Comments

@berak
Copy link
Contributor

@berak berak commented Dec 7, 2018

  • OpenCV =>3.4.3 / 3.4.4
  • Operating System / Platform => Windows 64 Bit / ubuntu 16
  • Compiler => opencv-contrib-python 3.4.4 / 3.4.3 (from pip)

for unknown reason, sample only runs,
if we use cv.reg_MapShift cv.reg_MapperGradShift and cv.reg_MapperPyramid here:

mapTest = cv.reg.MapShift(shift)
img2 = mapTest.warp(img1)
mapper = cv.reg.MapperGradShift()
mappPyr = cv.reg.MapperPyramid(mapper)

(underscores instead of dots)
(easy to fix, but the underscores look quite weird to me)

it also segfaults, if it used like this:

 mappPyr = cv.reg_MapperPyramid(cv.reg_MapperGradShift()) 

instead of:

 mapper = cv.reg_MapperGradShift() 
 mappPyr = cv.reg_MapperPyramid(mapper) 
@lostcontrol
Copy link

@lostcontrol lostcontrol commented Jun 12, 2019

I can confirm this with:

opencv-contrib-python==4.1.0.25
opencv-python==4.1.0.25

@SunilKrSingh
Copy link

@SunilKrSingh SunilKrSingh commented Dec 6, 2019

Hi @lostcontrol
I am new to Open Source. Have worked on only personal projects. I want to solve this issue.
Can you please tell me what exactly I have to do here? Thanks.

P.S. I have used OpenCV before and I am proficient in python.

@lostcontrol
Copy link

@lostcontrol lostcontrol commented Dec 23, 2019

@SunilKrSingh, I'm not involved in OpenCV but have a look at the first comment and check the sample code. Based on that you should be able to reproduce the issue. Good luck.

@supertramp2
Copy link

@supertramp2 supertramp2 commented Apr 4, 2020

@alalek This issue has been solved already right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.