Skip to content
Snippets Groups Projects

Update ImageClip and ImageRegion

Closed Ghost User requested to merge (removed):master into master
2 unresolved threads
  • use parallel projection camera instead of perspective camera to avoid inaccurate xy positions caused by different z values of border widget and image actor

Merge request reports

Closed by avatar (Mar 13, 2025 8:17pm 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
  • Andrew Maclean
    • Resolved
      1. Including: #include <vtkCamera.h> will make the build work in both cases.
      2. Can you please modify the markdown files adding the reason for using parallel projection. In ImageRegion.md remove the last line since the image picking is working now.
      3. I am not seeing images similar to the test images for some reason.
        • In ImageClip I expected to see the selected region in the right renderer.
        • In ImageRegion I just see a green square in the upper right of the window.
  • Ghost User added 1 commit

    added 1 commit

    • d84cacd9 - Update ImageClip and ImageRegion

    Compare with previous version

  • Ghost User resolved all threads

    resolved all threads

    • Resolved

      You need to generate a new test image for ImageClip.cxx

      Go to your build folder for vtk-examples and run:

      ctest -R ImageClip

      Then replace the image in vtk-examples/src/Testing/Baseline/Cxx/Interaction/ with the new image.

  • Ghost User added 1 commit

    added 1 commit

    Compare with previous version

  • Ghost User resolved all threads

    resolved all threads

  • There seems to be an issue with this MR, you need to rune the merge request pipeline and fix any errors.

  • Ghost User added 39 commits

    added 39 commits

    Compare with previous version

  • A deleted user closed

    closed

  • Please register or sign in to reply
    Loading