From 1254687132c19811e6c35d74f73193244d26d093 Mon Sep 17 00:00:00 2001 From: rnc Date: Mon, 9 Dec 2013 16:29:54 +0000 Subject: [PATCH] Changed a default value for Contour detection feature --- src/EntityGUI/EntityGUI_FeatureDetectorDlg.cxx | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/src/EntityGUI/EntityGUI_FeatureDetectorDlg.cxx b/src/EntityGUI/EntityGUI_FeatureDetectorDlg.cxx index de72d17c3..96bbb34fe 100644 --- a/src/EntityGUI/EntityGUI_FeatureDetectorDlg.cxx +++ b/src/EntityGUI/EntityGUI_FeatureDetectorDlg.cxx @@ -282,7 +282,16 @@ EntityGUI_FeatureDetectorDlg::EntityGUI_FeatureDetectorDlg( GeometryGUI* theGeom myWidgets.insert( HIST_TYPE, histType ); QDoubleSpinBox* thresholdValue = new QDoubleSpinBox(); thresholdValue->setRange( 0, 254 ); - thresholdValue->setValue( 128 ); + // 1 is a good default value for the threshold. It means that we are very permissive + // about what will be considered INSIDE the zone we want to find the frontier of + // This makes the algorithm more robust against a bit inhomogeneous parts in the zone + // that we want to delimitate. + // The drawback is if we want to delimitate a zone wich color is very similar to the zone + // we consider as the OUTSIDE, the result will be bad. + // The current use cases are more of the first form : + // - Strongly contrasted INSIDE and OUTSIDE zones + // - Small inhomogenities in each zone + thresholdValue->setValue( 1 ); myWidgets.insert( THRESHOLD_VALUE, thresholdValue ); QDoubleSpinBox* maxThreshold = new QDoubleSpinBox(); maxThreshold->setRange( 1, 255 ); -- 2.39.2