Skip to content
GitLab
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 204
    • Issues 204
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 11
    • Merge requests 11
  • 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
  • !861

Fixes in Sar metadata IOs

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Cédric Traizet requested to merge sar_io into develop Oct 08, 2021
  • Overview 0
  • Commits 5
  • Pipelines 3
  • Changes 5

Summary

This merge request fixes a few issues related to SAR metadata reading and writing:

  • A fix for the parsing of azimuth range noise vector in S1 product, when there is only one vector (e.g. IW SLC products)
  • The GCPs are now written as metadata alongside the SARParam metadata, because a SAR sensor image might have a geotransform, but GDAL can't handle GCP and geotransform. The written GCPs are left unmodified, and the geotransform should be used to locate them (this is the same behavior as the one used for the GCPs in OssimSarSensorModel). methods to convert GCP and GCPParam from and to keywordlist have been added.
  • SAR calibration metadata (SARCalib) is now exported in GDALImageIO

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
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: sar_io