Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • otb otb
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 184
    • Issues 184
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 9
    • Merge requests 9
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Main Repositories
  • otbotb
  • Merge requests
  • !484

Remove docname

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Victor Poughon requested to merge remove-docname into develop Apr 29, 2019
  • Overview 2
  • Commits 3
  • Pipelines 6
  • Changes 129

Summary

Remove DocName from applications:

  • Remove data member from WrapperApplication class, remove getters/setters from C++ and python APIs.
  • Remove SetDocName from all applications
  • Show only Name in documentation
  • Remove column from GUI application browser
  • Fix python API Application.__str__()

Rationale

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).

Copyright

The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement.


Check before merging:

  • All discussions are resolved
  • At least 2 👍 votes from core developers, no 👎 vote.
  • The feature branch is (reasonably) up-to-date with the base branch
  • Dashboard is green
  • Copyright owner has signed the ORFEO ToolBox Contributor License Agreement
  • Optionally, run git diff develop... -U0 --no-color | clang-format-diff.py -p1 -i on latest changes and commit
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: remove-docname