There is a significant amount of duplicated code between otbWrapperQtWidgetIntParameter and otbWrapperQtWidgetFloatParameter. This is not addressed by this MR. Unfortunately I think that the fix is more complex than simply moving the common code to a base class. Might be possible with multiple inheritance of an interface common to QtWidgetSpinBox and QtWidgetDoubleSpinBox.
The new float widget does not show decimal points for float values that are also integers. This may confuse users who will not figure out that they are allowed to input non-integer values. Similarly, because the new LineEdit allows all input and only clears it when focus is lost, users might enter decimal values in int parameter, only to be confused why their input is being truncated.
As a fundamental limitation of IEEE float, there will be loss of precision (and therefore weird GUI behavior) beyond 6 decimal digits in float inputs. It is not possible to go around this without also changing application engine's floats to doubles.
Copyright
The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement
Victor Poughon authored