Skip to content
Snippets Groups Projects

Fix StreamingWarpImageFilter issue with empty region

Merged Julien Michel requested to merge 1963-fix-streamingwarpimagefilter into develop
All threads resolved!

Summary

Closes #1963 (closed)

Add a test to demonstrate the issue (currently failing).

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
  • Optionally, run git diff develop... -U0 --no-color | clang-format-diff.py -p1 -i on latest changes and commit
Edited by Julien Michel

Merge request reports

Merge request pipeline #2591 passed

Merge request pipeline passed for 71a08619

Approval is optional

Merged by Julien MichelJulien Michel 5 years ago (Sep 13, 2019 12:58pm UTC)

Merge details

Pipeline #2599 passed

Pipeline passed for f4b9516d on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Julien Michel resolved all threads

    resolved all threads

  • Julien Michel added 2 commits

    added 2 commits

    • 86fabd1f - COMP: attempt to fix windows builds
    • 71e3dcb4 - REFAC: Small refactoring to make code easier to read (from PR review)

    Compare with previous version

  • Julien Michel resolved all threads

    resolved all threads

  • Julien Michel added 2 commits

    added 2 commits

    • 6b2475b7 - COMP: attempt to fix windows builds 2/2
    • 71a08619 - COMP: Attempt to fix fast build

    Compare with previous version

  • Author Reporter

    CI ok, need votes to merge.

  • Julien Michel mentioned in commit f4b9516d

    mentioned in commit f4b9516d

  • merged

  • Please register or sign in to reply
    Loading