-
Notifications
You must be signed in to change notification settings - Fork 0
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
ENH: Add OpenXRRemoting external project #2
Conversation
Download v2.9.1 of Microsoft.Holographic.Remoting.OpenX References: * https://www.nuget.org/packages/Microsoft.Holographic.Remoting.OpenXr/2.9.1
648006e
to
43c3a46
Compare
Notes on installing Visual Studio components to support building the "player" application referenced in the See https://discourse.slicer.org/t/windows-preview-build-delayed-visual-studio-2022-updates/29045 Footnotes |
Step by step
Building the player appReferences
PrerequisitesUpdate Visual Studio 2022 installation (Tools -> Get Tools and Features):
To address the error message "Sorry, something went wrong"`, see these instructions. BuildingOn Windows, build the "player application"
Open On first use, right-click the solution and select Select |
…ctivate Update vtkSlicerMixedRealityLogic: * Add SetMixedRealityConnected/GetMixedRealityConnected * Add SetMixedRealityActive/GetMixedRealityActive * SetDefaultReferenceView
071a631
to
30541a4
Compare
Fixes: ``` Exception thrown: read access violation. this was nullptr. Call stack: vtkRendering-9.1.dll!vtkRenderer::SetActiveCamera(vtkCamera * cam) Line 770 C++ ```
When using remoting, the OpenXR loader must be built dynamically to allow the remoting package to override this loader at runtime. (see RemotingXR.json) Fixes: ``` ERROR [GENERAL | xrGetInstanceProcAddr | OpenXR-Loader]: No active xrInstance handle. ```
Fixes: Unable to open openXR action file : [...]/externals/vtkRenderingOpenXRvtk_openxr_actions.json
This requires VTK MR-10449 (see [1]) backported to Slicer/VTK through the Slicer PR-7190 (see [2]). [1] https://gitlab.kitware.com/vtk/vtk/-/merge_requests/10449 [2] Slicer/Slicer#7190
Pull requests implementing required changes: |
Ensure that the "openxr_loader.dll" library is found when loading the MixedReality module from a build tree using the "SlicerWithSlicerMixedReality.exe" launcher.
Pull request implementing useful changes: |
VTK_INSTALL_RUNTIME_DIR or VTK_INSTALL_LIBRARY_DIR CMake variables were completely removed in VTK 9.1.
List of changes: $ git shortlog 10d08094..b03e854a4 --no-merges Jean-Christophe Fillion-Robin (3): Add SuperBuild support Remove redundant BUILD_TESTING option COMP: Support configuring install directories
Adapted from KitwareMedical/SlicerVirtualReality@a4d465b73 Remove the "externals" sub-directory that was specific to branch Slicer/[email protected] integrating commit Slicer/VTK@0ba7af630 ([Backport MR-8123] vtk_module_build: Ensure external modules build directory are not overwritten) Since the approach originally proposed in VTK MR-8123 was superseded by MR-9097 there is no need to specify the "externals" sub-directory.
Pull request implementing required changed: Especially the backport of MR-10452: |
fa3d863
to
ac01e64
Compare
Note that the vtkMixedRealityViewInteractorStyle still derives from vtkOpenXRInteractorStyle whereas the VirtualReality counterpart derives from vtkMRMLViewInteractorStyle.
290c64c
to
45a8513
Compare
a4685f5
to
c8cdf68
Compare
The extension can now be built on Windows (and even packaged & installed) without requiring the developer to copy files and libraries. Next step we will need to refactor We will also have to improve the OpenXR integration so that we can retrieve the position of each controllers & trackers: |
Download v2.9.1 of
Microsoft.Holographic.Remoting.OpenXR
References: