Skip to content

Review applications parameter order

Victor Poughon requested to merge apps_parameter_order into develop

Summary

One common confusion from users that I see when doing OTB workshops is that some applications' parameters are all the way at the bottom of the parameters list. This is not a problem in cli but shows up in gui. For example EdgeExtraction:

Screenshot_from_2018-10-25_16-38-07

This MR changes the order of a few parameters to roughly follow the convention: start with mandatory parameters related to I/O. For example EdgeExtraction becomes:

Screenshot_from_2018-10-25_16-27-57

Rationale

Better otbgui UX. User intuition is "higher parameters are more important".

Implementation Details

Depends on !273 (merged) to avoid a git conflict. I will change the target branch of this MR to develop when it's merged.

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
Edited by Victor Poughon

Merge request reports