Skip to content

Remove unnecessary segmentation faults:#3782

Closed
hziSot wants to merge 1 commit intoopencv:masterfrom
hziSot:master
Closed

Remove unnecessary segmentation faults:#3782
hziSot wants to merge 1 commit intoopencv:masterfrom
hziSot:master

Conversation

@hziSot
Copy link
Copy Markdown

@hziSot hziSot commented Mar 4, 2015

$ ./bin/opencv_createsamples -w
zsh: segmentation fault (core dumped) ./bin/opencv_createsamples -w
$ ./bin/opencv_traincascade -numThreads
zsh: segmentation fault (core dumped) ./bin/opencv_traincascade -numThreads

$ ./bin/opencv_createsamples -w
zsh: segmentation fault (core dumped)  ./bin/opencv_createsamples -w
$ ./bin/opencv_traincascade -numThreads
zsh: segmentation fault (core dumped)  ./bin/opencv_traincascade -numThreads
mkostin pushed a commit to microsoft/opencv that referenced this pull request Mar 4, 2015
@mshabunin
Copy link
Copy Markdown
Contributor

@hziSot, I think it will be better to use cv::CommandLineParser class instead. It is not well documented, but there are many examples:

What is your opinion? Can you do it?

@hziSot
Copy link
Copy Markdown
Author

hziSot commented Mar 4, 2015

Will try.
Should I close this pull request and create the new one?

@mshabunin
Copy link
Copy Markdown
Contributor

Thank you, I'll provide help if you need.

Should I close this pull request and create the new one?

Usually it is not needed, but this case is slightly different: our integration system can have problems when merging with branches named master and 2.4. So, the answer is: yes, please, close this pull request and create another one from branch with different name (for example fix-apps-cmd-parse).

@StevenPuttemans
Copy link
Copy Markdown

👍 nice fix! Love to see more people get involved with the cascade classifier interface :) Seems still heck a lot of bugs are out there :D

@mshabunin mshabunin closed this Mar 17, 2015
@mshabunin mshabunin removed their assignment Oct 12, 2015
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.

3 participants