Skip to content

Remove convert app

Cédric Traizet requested to merge remove_convert_app into develop

Summary

This merge request removes the Convert application.

Rationale

Convert is deprecated and will be removed in OTB 7.0.

Implementation Details

The following modifications have been made :

  • The Convert application has been removed from the AppImageUtils/app CMakeLists and the otbConvert.cxx has been deleted
  • The extended filename tests apTuUtConvertExtendedFilename_writer and apTvUtConvertExtendedFilename_readerGEOM now use DynamicConvert instead of Convert. They have been renamed in apTuUtDynamicConvertExtendedFilename_writer and apTvUtDynamicConvertExtendedFilename_readerGEOM. The baseline is unchanged. Note that those are the only test in OTB using the extended filename mechanism in an application.
  • The other tests using Convert have been removed, along with their baseline.
  • Convert was used in the cookbook as an example application (for otbcli, otbgui and extended filenames). It has been replaced by DynamicConvert.
  • Convert has been removed from the SetDocSeeAlso method of applications ConcatenateImages and DynamicConvert
  • The python test Bug440 has been removed, as it tests the same thing as, for instance, PythonRescaleTest. This test was using the Convert application.

Additional notes

Convert is replaced by DynamicConvert (https://www.orfeo-toolbox.org/CookBook/Applications/app_DynamicConvert.html)

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 Cédric Traizet

Merge request reports