Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: Update VTK backporting OpenVR rendering fix when HMD inside the volume #7507

Conversation

jcfr
Copy link
Member

@jcfr jcfr commented Dec 23, 2023

This addresses proper rendering, particularly by vtkOpenGLGPUVolumeRaycastMapper, when the head-mounted display (HMD) is positioned inside the volume. The issue is resolved by ensuring correct computation of the plane frustum when the camera is tracking the HMD and is positioned inside the volume.

The changes listed below impact the following VTK modules:

  • Rendering/VR
  • Rendering/OpenVR
  • Rendering/OpenXR

Fixes:

For more details:

List of VTK changes:

$ git shortlog c69cf77a6..7767cabe4 --no-merges
Sankhesh Jhaveri (3):
      [Backport MR-10783] VR: Move the TrackHMD flag to the camera
      [Backport MR-10783] VR: Deprecate vtkVRRenderWindow::TrackHMD
      [Backport MR-10783] VR: Provide stereo projection transforms for rendering

cc: @sankhesh @LucasGandel

…volume

This addresses proper rendering, particularly by `vtkOpenGLGPUVolumeRaycastMapper`,
when the head-mounted display (HMD) is positioned inside the volume. The issue is
resolved by ensuring correct computation of the plane frustum when the camera is
tracking the HMD and is positioned inside the volume.

The changes listed below impact the following VTK modules:
* `Rendering/VR`
* `Rendering/OpenVR`
* `Rendering/OpenXR`

List of VTK changes:

```
$ git shortlog c69cf77a6..7767cabe4 --no-merges
Sankhesh Jhaveri (3):
      [Backport MR-10783] VR: Move the TrackHMD flag to the camera
      [Backport MR-10783] VR: Deprecate vtkVRRenderWindow::TrackHMD
      [Backport MR-10783] VR: Provide stereo projection transforms for rendering
```

Co-authored-by: Sankhesh Jhaveri <sankhesh.jhaveri@kitware.com>
Co-authored-by: Lucas Gandel <lucas.gandel@kitware.com>
@jcfr jcfr force-pushed the update-vtk-fix-plane-frustum-for-vr-rendering-volume-inside branch from a658821 to 6448c27 Compare December 23, 2023 07:10
@jcfr
Copy link
Member Author

jcfr commented Dec 23, 2023

Since the change has been tested locally and confirmed to fix issue KitwareMedical/SlicerVirtualReality#125 ✅, moving forward with the integration 🚀

@jcfr jcfr added the backport:5.x Identify pull request expected to be backported to the current 5.x release branch. label Dec 23, 2023
@jcfr jcfr merged commit 1ba1b03 into Slicer:main Dec 23, 2023
6 checks passed
@jcfr jcfr deleted the update-vtk-fix-plane-frustum-for-vr-rendering-volume-inside branch December 23, 2023 07:14
@jcfr jcfr removed the backport:5.x Identify pull request expected to be backported to the current 5.x release branch. label Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant