An error occurred while fetching the assigned milestone of the selected merge_request.
Remove docname
Compare changes
Remove DocName
from applications:
WrapperApplication
class, remove getters/setters from C++ and python APIs.SetDocName
from all applicationsName
in documentationApplication.__str__()
For 95% of OTB applications, the extra doc name is useless. Except for maybe those three applications:
LSMSSegmentation Exact Large-Scale Mean-Shift segmentation, step 2
LSMSSmallRegionsMerging Exact Large-Scale Mean-Shift segmentation, step 3 (optional)
LSMSVectorization Exact Large-Scale Mean-Shift segmentation, step 4
the DocNames are same as the normal Names, but with spaces, casing and word order changed. The DocName adds nothing to the normal Name. Additionally it can be confusing for users when mixing cli/gui interfaces and reading documentation.
For simplification I propose to remove it, but we could also add a dummy implementation of GetDocName/SetDocName
that does nothing, to improve backwards compatibility with < 7.x.
Closes #1789 (closed).
The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement.
Check before merging:
git diff develop... -U0 --no-color | clang-format-diff.py -p1 -i
on latest changes and commit