Skip to content
Snippets Groups Projects

PERF: Improve FunctorImageFilter performances

Merged Luc Hermitte requested to merge lhermitte/otb:line_progress into develop

Summary

Improve slightly the performances of otb::ImageFunctorFilter.

Rationale

otb::ImageFunctorFilter with a lambda is slightly less performant than itk::UnaryFunctorImageFilter with a dynamic functor.

Implementation Details

itk::UnaryFunctorImageFilter only updates the progress reporter once per line.

This MR mimics ITK behaviour.

Results

The performance improvement is small, less than 10% on simple filters like a scaling filter that multiplies pixel values. The gap seems to have been filled.

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

Merge request reports

Merge request pipeline #378 passed

Merge request pipeline passed for 52fdb672

Merged by Victor PoughonVictor Poughon 6 years ago (Mar 12, 2019 8:43am UTC)

Merge details

  • Changes merged into with b28c5338.
  • Deleted 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
Please register or sign in to reply
Loading