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 189
    • Issues 189
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 10
    • Merge requests 10
  • 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
  • !776

Improve tests prTvTestCreateInverseForwardSensorModel

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Julien Osman requested to merge improve_tests_InverseForwardSensorModel into develop Nov 25, 2020
  • Overview 0
  • Commits 2
  • Pipelines 5
  • Changes 5

Summary

Improve the tests prTvTestCreateInverseForwardSensorModel_Toulouse and prTvTestCreateInverseForwardSensorModel_Cevennes so it compares return values.

Rationale

The tests for the otb::SensorModel classes were removed years ago, because the they depended on the precision in OSSIM, that could change with OSSIM's version. This was fine at this time, because the values were validated in OSSIM, and the OTB part was tested in the filters. But we are introducing big changes with the remove OSSIM story, and tests closer to the changes are helpful to track down eventual regressions. We set a big tolerance on the values, so little changes of prevision in the dependencies wont affect this tests.

Implementation Details

Tests

Add the possibility to set an optional DEM. Add an inverse transform and a forward transform of a point in the image, to assert the quality of the transformation.

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 👍 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 Dec 01, 2020 by Julien Osman
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: improve_tests_InverseForwardSensorModel