Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
otb
otb
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 298
    • Issues 298
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 15
    • Merge Requests 15
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Main Repositories
  • otbotb
  • Merge Requests
  • !727

Merged
Opened Jul 22, 2020 by Rémi Cresson@remicressOwner

FIX: StreamingMosaicFilter childs can have a different number of components per pixel at output

  • Overview 2
  • Commits 4
  • Pipelines 6
  • Changes 2

Summary

StreamingMosaicFilterBase throws some warning and discard no-data pixels provided by the user, when the number of component per pixel (NCPP) is not the same in input images and output images. This patch removes this behavior, enabling user to chose a no-data pixel with different NCPP in the output image.

Rationale

Mosaic filters derive from StreamingMosaicFilterBase. Most of the mosaic logic remains in this class, and the deriving filters basically just implement how the functor is applied to pixels. Before this patch, StreamingMosaicFilterBase assumed that the NCPP of the output mosaic image, is the same as the input images NCPP. While this is mostly true in the actual existing StreamingMosaicFilterXXX filters, one could imagine some filters that need to change the output number of components per pixels (for instance, that one computes the argmax function of the averaged input channels mosaic).

Implementation Details

Just remove the warning and not reset the no-data output pixel when its NCCP is different than input.

Classes and files

Only StreamingMosaicFilterBase.hxx is modified.

Applications

No change in applications, because the no-data pixel is the same in input and output anyway.

Copyright

The copyright owner is IRSTEA (Rémi Cresson) 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
Reviewer
Request review from
7.2.0
Milestone
7.2.0
Assign milestone
Time tracking
Reference: orfeotoolbox/otb!727
Source branch: mosaic_fix_nb_channels