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 184
    • Issues 184
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 9
    • Merge requests 9
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • 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
  • !831

Merged
Created Jul 22, 2021 by Cédric Traizet@ctraizetDeveloper

Fix thermal noise removal in Sentinel 1 calibration

  • Overview 4
  • Commits 8
  • Pipelines 8
  • Changes 12

Summary

This merge request fix two issues in the thermal noise:

  • For VH products, the VV LUTs were read by ossimplugins instead of the VH LUTs (calibration LUT and noise), resulting in error in the denoising. For the calibration, it seems that the LUT are identical for VV and VH products (is it always the case?).
  • There was a cast from double to int, causing small errors in the computed noise.

The results have been validated by hand for a few points, and on a whole product by comparing the output produced by the SARCalibration application with the calibration produced by snap. Some differences have been noticed between the outputs of the two software, but there are likely caused by a bug in SNAP

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 Jul 29, 2021 by Cédric Traizet
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: noise_s1_test