Skip to content
Snippets Groups Projects

Enhance StreamingStatisticsMapFromLabelImageFilter

Merged Rémi Cresson requested to merge enhance_mapfromlabelstats into develop
All threads resolved!

Summary

Improves the otbStreamingStatisticsMapFromLabelImageFilter with the following enhancements:

  • Compute count, mean, standard deviation, min, max
  • Multithreaded

Rationale

A class named StatisticsAccumulator handles the update of statistics. I feel that in a near future refactoring we could create such container(s) for statistics. This way, we could reuse this in many otb image statistics components (StreamingImageStatistics and co.). Moreover, we could enrich the persistent filter base classes to enable the (optional) native support of MPI. (But of courses this deserve a complete new RFC)

Implementation Details

Classes and files

Give an overview of the implementation: main changes made to classes, files and modules. Do not paste complete diff, as it is available in the merge request already.

Applications

Describe any changes made to existing applications, or new applications that have been added.

Tests

Describe the testing strategy for new features.

Documentation

List or link documentation modifications that were made (doxygen, example, Software Guide, application documentation, CookBook).

Additional notes

List remaining open issues if any, and additional notes.

Copyright

The copyright owner is IRSTEA and has signed the ORFEO ToolBox Contributor License Agreement

Edited by Rémi Cresson

Merge request reports

Merged by Victor PoughonVictor Poughon 6 years ago (Jul 18, 2018 9:51am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Rémi Cresson resolved all discussions

    resolved all discussions

  • Rémi Cresson added 1 commit

    added 1 commit

    • 37dc36c0 - REFAC: rename accumulatorMaps to m_AccumulatorMaps

    Compare with previous version

  • Rémi Cresson resolved all discussions

    resolved all discussions

  • Manuel Grizonnet
  • Rémi Cresson added 1 commit

    added 1 commit

    Compare with previous version

  • added patch label

  • Manuel Grizonnet mentioned in merge request !62 (closed)

    mentioned in merge request !62 (closed)

  • Luc Hermitte
  • Luc Hermitte
  • Luc Hermitte
  • Luc Hermitte
  • Luc Hermitte
  • Rémi Cresson added 1 commit

    added 1 commit

    • 229f72e6 - REFAC: address Luc's comments

    Compare with previous version

  • Rémi Cresson added 354 commits

    added 354 commits

    Compare with previous version

  • Rémi Cresson added 1 commit

    added 1 commit

    • 6ed654b1 - FIX: fix failing test on dashboard

    Compare with previous version

  • removed patch label

  • @remicress I've removed the patch label from this merge request as it is for small patch which do not imply to test them on the dashboard.

    The dashboard concerning this MR is good apart from warning on Windows (also have a look to hulk platform maybe). I think that they should be fixed:

    https://dash.orfeo-toolbox.org/viewBuildError.php?type=1&buildid=325179

    So after this you just have to update the branch against develop and you'll be ready to merge this one (you've got 2 +1)!

  • Rémi Cresson added 1 commit

    added 1 commit

    Compare with previous version

  • Rémi Cresson added 6 commits

    added 6 commits

    Compare with previous version

  • Rémi Cresson added 5 commits

    added 5 commits

    Compare with previous version

  • How can I fix those warning with the [[fallthrough]] thing?

  • @remicress . [[falltrough]] is ignored in C++11 and 14, and taken into account by the compilers from C++17 onward. Except that it seems that VC++ emits a warning telling us that it's not recognized yet :( My mistaken then.

    Two choices:

    1. we can remove it
    2. we can add an option into the CMakeLists.txt files for VC++ case to tell them to silence C5030 warning. Note that it can also be done with pragmas: https://stackoverflow.com/questions/7159348/disable-single-warning-error
  • I would say that for now we can't use c++17 features so probably better to remove it.

  • Victor Poughon resolved all discussions

    resolved all discussions

  • Rémi Cresson resolved all discussions

    resolved all discussions

  • Rémi Cresson added 2 commits

    added 2 commits

    • 16d0f0a3 - REFAC: replace vcl_sqrt with std::sqrt
    • 615ce5ed - REFAC: remove useless switch for output type

    Compare with previous version

  • Please add the copyright owner to the MR

  • Rémi Cresson changed the description

    changed the description

  • Status: we are waiting for @remicress CLA and will be go for merge as soon as we receive it

  • Victor Poughon resolved all discussions

    resolved all discussions

  • Victor Poughon added 97 commits

    added 97 commits

    Compare with previous version

  • We have received Rémi's CLA.

    I just fixed the conflict with develop. Let's wait tomorrow dashboard and merge if green.

    Edited by Victor Poughon
  • Victor Poughon mentioned in commit a7d3e7ec

    mentioned in commit a7d3e7ec

  • mentioned in issue #1722 (closed)

  • mentioned in issue #1682 (closed)

  • Please register or sign in to reply
    Loading