Skip to content
Snippets Groups Projects

ENH : Add metadata to Sentinel 1 Model

Merged ENH : Add metadata to Sentinel 1 Model
ussegliog/otb:addMetadata_ossimSentinel1 into develop
All threads resolved!
Merged Gaëlle USSEGLIO requested to merge ussegliog/otb:addMetadata_ossimSentinel1 into develop
All threads resolved!

Summary

New Metadata for Sentinel 1 Model

Rationale

The metadata will be used by new processing such as deramping/reramping.

Implementation Details

Classes and files

Four classes was changed :

  • ossimSarSensorModelPathsAndKeys : New keys
  • ossimSarSensorModel : Spread BurstRecordType
  • ossimSentinel1Model :
    • Changement into read function to add or change some metadata. For instance, the pulse_repetition_frequency is now equivalent to azimuthFrequency (from annotation.xml file) and azimuth_steering_rate was added into geom file.
    • Changement into addDopplerCentroidCoefficients
    • New function : addAzimuthFmRateCoefficients
    • New prefix for metadata from manifest file
    • Support_data tags contain only information from annotation.xml file
    • Header version was incremented by one (third version of geom file for Sentinel 1 Model).
  • ossimSentinel1SarSensorModel : Not used (apparently)

NB : The version of geom file can be checked by some applications and an exeception can be thrown if the third version (at least) is required.

Tests

The application ReadImageInfo was used to visualize the new geom file for Sentinel 1 data.

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
Edited by Gaëlle USSEGLIO

Merge request reports

Approval is optional

Merged by Victor PoughonVictor Poughon 6 years ago (Dec 4, 2018 4:36pm UTC)

Merge details

  • Changes merged into develop with 33f5066a.
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Luc Hermitte
  • Victor Poughon
  • Gaëlle USSEGLIO added 1 commit

    added 1 commit

    • 010590c6 - ENH : std::stoi instead of atoi

    Compare with previous version

  • Julien Michel
    Julien Michel @jmichel ·
    Reporter

    This needs to be added for dashboard testing.

  • Julien Michel
    Julien Michel @jmichel ·
    Reporter

    Added branch for testing.

  • Gaëlle USSEGLIO added 1 commit

    added 1 commit

    • 162212ae - BUG : Correction into ossimSarSensorModel to get/read new elements only if header version >= 3

    Compare with previous version

  • Gaëlle USSEGLIO changed the description

    changed the description

  • Gaëlle USSEGLIO resolved all discussions

    resolved all discussions

  • Gaëlle USSEGLIO added 1 commit

    added 1 commit

    • 34ae0984 - WRG : Correction for Code review into Sentinel1SarSensorModel

    Compare with previous version

  • Victor Poughon
    Victor Poughon @poughov ·
    Contributor

    This is go for merge as soon as dashboard is green with the latest commits

  • Victor Poughon merged

    merged

  • Victor Poughon mentioned in commit 33f5066a

    mentioned in commit 33f5066a

  • Victor Poughon
    Victor Poughon @poughov ·
    Contributor

    merged! Thanks @ussegliog

  • Cédric Traizet added feature label

    added feature label

  • Cédric Traizet changed milestone to %7.0.0

    changed milestone to %7.0.0

    • Please register or sign in to reply