stereosgbm - doing XSOBEL and intensity simultaneously does not make sense (Feature #358)


Added by Stefan Hahn almost 15 years ago. Updated about 12 years ago.


Status:Cancelled Start date:
Priority:Normal Due date:
Assignee:Vadim Pisarevsky % Done:

0%

Category:calibration, 3d
Target version:-
Difficulty: Pull request:

Description

This is my first ticket for opencv. I hope it is reasonable.

Recently, an XSOBEL filter was added to cvstereosgbm.cpp.

In the literature this filter has been mainly used to remove intensity bias between the image stereo pairs.

If you look at how XSOBEL is used in opencv, especially the line

for( c = 0; c < cn*2; c++, prow1 += width, prow2 += width )

You can see (and debug) that the BT filtered intensity difference costs are added onto the cost cube ALS WELL AS the XSOBEL (and BT) filtered difference costs. This means, you do not get rid of the intensity bias but have just a little "vertical edge enhancement".

I assume it only makes sense to do either pure or XSOBEL filtered intensity. This would either require an additional bool variable or - in the sense of Donald Knuth - to leave XSOBEL out of the SGM routine and build a separate XSOBEL filter in opencv. I could also imagine more filters as the stereo comunity is testing a whole variety of preprocessing filters at the moment. (I would like a 7x9 census, which is the current favourite in the community - which unfortunately requires to modify the pixel difference operation within cvstereosgbm as well.)


Associated revisions

Revision 86b4b30a
Added by Andrey Kamaev about 12 years ago

Merge pull request #358 from taka-no-me:objc_exceptions

History

Updated by Victor Eruhimov over 14 years ago

  • Status deleted (Open)

Updated by Alexander Shishkov almost 13 years ago

  • Status set to Open

Updated by Vadim Pisarevsky about 12 years ago

ok, closing it, since we will not be doing anything with it

  • Status changed from Open to Cancelled

Also available in: Atom PDF