Skip to content
GitLab
Projects Groups Topics 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
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 207
    • Issues 207
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 12
    • Merge requests 12
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and 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
  • !613

Additional tests for the ExtractROI application

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Cédric Traizet requested to merge test_extractroi into develop Oct 16, 2019
  • Overview 6
  • Commits 11
  • Pipelines 12
  • Changes 9

Summary

Currently, the ExtractROI application is only tested in the case where the output is the same as the input, for example if its parameters are not compliant with the input image. it is also important to test the case where the application works correctly. This MR adds the 9 following tests:

  • Mode standard (test with sizex sizey startx and starty) + cl parameter
  • Mode Fit:
    • case with a reference image
    • case with a reference vector
  • Mode Extent (test with mode.extent.ulx, mode.extent.uly ,mode.extent.lrx and mode.extent.lry:
    • case with unit = pxl
    • case with unit = phy
    • case with unit = lonlat
  • Mode Radius (test with radius.r radius.cs and radius.cy :
    • case with center unit pxl and radius unit pxl
    • case with center unit phy and radius unit phy
    • case with center unit lonlat and radius unit pxl

This MR also corrects a bug in fit vector mode, and in the extent mode with physical units, where some variable were redefined in a scope.

Closes issue #1931 (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
Edited Oct 16, 2019 by Cédric Traizet
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: test_extractroi