Possibly misleading/incorrect bibliography entry (Feature #1627)


Added by Emily Cooper about 13 years ago. Updated almost 13 years ago.


Status:Done Start date:2012-02-28
Priority:Low Due date:
Assignee:Vadim Pisarevsky % Done:

0%

Category:documentation
Target version:2.4.0
Difficulty: Pull request:

Description

The source for the cost function used by StereoSGBM is given as Birchfield & Tomasi (1996) "Depth Discontinuities by Pixel-to-Pixel Stereo." This is the same source suggested in Hirshmuller (2008) as an alternative to using Mutual Information.

While the Birchfield & Tomasi (1996) paper does describe sampling insensitive sub-pixel dissimilarity, the primary purpose of the paper is to introduce a more specific cost function this is specific for detecting depth discontinuities. This cost function includes additional terms that penalize discontinuities in depth, making it less sensitive to small depth variations within a object.

I believe the cost function that is actually implemented in StereoSGBM does not include these additional discontinuity terms, so it is misleading to cite the 1996 paper. A more correct reference would be Birchfield & Tomasi (1998) "A Pixel Dissimilarity Measure that is Insensitive to Image Sampling," which describes the simpler cost function in detail.

If StereoSGBM does in fact use the cost function that is specialized for depth discontinuities, it would be good for this to be noted in the documentation.


BT1998.pdf (824.6 kB) Emily Cooper, 2012-02-28 01:45 am

BT1999_1996_.pdf (940.8 kB) Emily Cooper, 2012-02-28 01:45 am


Associated revisions

Revision 6e5e2aa3
Added by Vadim Pisarevsky almost 13 years ago

fixed references in calib3d (including the ticket #1627)

Revision 572c18da
Added by Roman Donchenko over 11 years ago

Merge pull request #1627 from SpecLad:java-version-no-inline

History

Updated by Alexander Shishkov almost 13 years ago

  • Target version deleted ()

Updated by Alexander Shishkov almost 13 years ago

  • Priority changed from Normal to Low

Updated by Alexander Shishkov almost 13 years ago

  • Tracker set to Feature

Updated by Alexander Shishkov almost 13 years ago

  • Target version deleted ()

Updated by Vadim Pisarevsky almost 13 years ago

thanks! the reference has been changed to the [BT98] paper in r7972

  • Status changed from Open to Done
  • Target version set to 2.4.0
  • Assignee set to Vadim Pisarevsky

Also available in: Atom PDF