Skip to content
Snippets Groups Projects

ENH: Allow to compute modulus and phase from 2 bands scalar image

Merged Manuel Grizonnet requested to merge modulus_2bands into develop
All threads resolved!

Summary

The support of complex images in otb applications as improved in version 6.6. There is generic mechanism based on teh ClampImageFilter which allow to go from/to complex or multi-bands scalar images. The only change is to read the input image as ComplexFloatImageType which support both complex image (single band) or image with 2 bands which corresponds to the real and imaginary parts. The documentation of the application has been updated to reflect this modification.

Rationale

Allow to compute modulus and phase from complex single image and 2 bands as it is fully supported by otb filters

Implementation Details

Applications

ComputeModulusAndPhase

Additional notes

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 :thumbsup: votes from core developers, no :thumbsdown: 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

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Manuel Grizonnet resolved all discussions

    resolved all discussions

  • added 1 commit

    • 5be2f62e - ENH: Use RegisterPipeline in ComputeModulusAndPhase application

    Compare with previous version

  • @aregimbeau thanks for the review, done.

  • Manuel Grizonnet added 5 commits

    added 5 commits

    Compare with previous version

  • Manuel Grizonnet added 4 commits

    added 4 commits

    Compare with previous version

  • mentioned in commit 3bceaa54

  • mentioned in issue #1682 (closed)

  • Please register or sign in to reply
    Loading