Skip to content
Snippets Groups Projects

Metadata reading from compound datasets

Merged Guillaume Pasero requested to merge 1589-support-s1-safe into develop
All threads resolved!

Summary

Improve the detection of metadata for sub-datasets ( SENTINEL1 SAFE, ...)

Rationale

It was implemented to solve #1589 (closed).

Implementation Details

Classes and files

In a nutshell, we can use the list of files attached to a dataset (see GDALDataset::GetFileList() ) as a final step when looking for OSSIM sensor models.

This list is stored in otb::ImageIOBase, empty by default. Only GDALImageIO can fill it for now. The ImageFileReader will try this list when no keywordlist has been found so far.

Tests

A tests has been added in OTBImageIO to verify the loading of a keywordlist from a VRT file.

Documentation

The SoftwareGuide has been updated (sensor model section).

Additional notes

The following use cases are now working:

Input path Keywordlist detected ?
S1A_IW_SLC_[...].SAFE Yes: IW1 image
S1A_IW_SLC_[...].SAFE/manifest.xml Yes: IW1 image
SENTINEL1_DS:S1A_IW_SLC_[...].SAFE:IW2_VV Yes: IW2 image
S1A_IW_SLC_[...].SAFE?sdataidx=2 Yes: IW3 image
iw1_vv_vh.vrt (stack of VV and VH) Yes: IW1 VV image

As shown, it also works with VRT files, it can load the metadata from the first raster source. This enables the orthorectification of sensor images in a VRT stack.

At the moment, any geom file associate with an attached file is ignored, but this could be added if there is a valid use case.

Copyright

The copyright owner is COPYRIGHT OWNER (OR OWNER'S AGENT) 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 Guillaume Pasero

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Manuel Grizonnet
  • added 1 commit

    • a7818f11 - DOC: clarify S1 Safe product

    Compare with previous version

  • Manuel Grizonnet resolved all discussions

    resolved all discussions

  • mentioned in commit 61c61e24

  • mentioned in issue #1589 (closed)

  • added bug label

  • Cédric Traizet changed milestone to %7.0.0

    changed milestone to %7.0.0

  • Please register or sign in to reply
    Loading