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 196
    • Issues 196
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 8
    • Merge requests 8
  • 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
  • !223

StreamingStatisticsMapFromLabelImageFilter no data support

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Laurențiu Nicola requested to merge streaming-statistics-map-nodata into develop Sep 07, 2018
  • Overview 11
  • Commits 4
  • Changes 2

Summary

  • add a progress reporter
  • reduce copying and lookups in otb::StreamingStatisticsMapFromLabelImageFilter
  • add support for ignoring pixels containing a certain value ("no data")

Rationale

When processing time series, it often happens that some values are missing. These changes make it possible to compute the various statistics while ignoring these no data values.

The statistics are still computed independently for each band, so a no data value in one of the bands does not exclude the pixel from the computation of the statistics for the bands.

Implementation Details

The three changes are separate commits to make reviewing easier.

Additional notes

I'm not sure if adding the itk::ProgressReporter is enough, no progress seemed to be displayed in my quick test.

Copyright

The copyright owner is CS ROMANIA / Laurențiu Nicola 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
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: streaming-statistics-map-nodata