Skip to content

OpenXR: Add passthrough demo#1273

Merged
Calinou merged 1 commit into
godotengine:masterfrom
BastiaanOlij:openxr-passthrough-demo
May 5, 2026
Merged

OpenXR: Add passthrough demo#1273
Calinou merged 1 commit into
godotengine:masterfrom
BastiaanOlij:openxr-passthrough-demo

Conversation

@BastiaanOlij

@BastiaanOlij BastiaanOlij commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

This PR adds a new OpenXR demo that shows the setup of passthrough (note that once we are happy with this demo, we should update that help page and direct it to the demo).

For now it's a bare bones demo just to show the basics but we can improve upon it.

The PR also makes sure all our copies of start_vr.gd are in sync.

@BastiaanOlij

Copy link
Copy Markdown
Contributor Author

IDK what to make of the CI failure...

Comment thread xr/openxr-passthrough-demo/main.gd Outdated
Comment thread xr/openxr_passthrough/start_vr.gd
Comment thread xr/openxr-passthrough-demo/start_vr.gd Outdated
Comment thread xr/openxr-passthrough-demo/README.md Outdated
Comment thread xr/openxr-passthrough-demo/README.md Outdated
Comment thread xr/openxr-passthrough-demo/README.md Outdated
Comment thread xr/openxr-passthrough-demo/README.md Outdated
Comment thread xr/openxr-passthrough-demo/README.md Outdated
@BastiaanOlij BastiaanOlij force-pushed the openxr-passthrough-demo branch 2 times, most recently from 2684eda to 4a37a1c Compare November 5, 2025 01:16
@BastiaanOlij

Copy link
Copy Markdown
Contributor Author

I've added a toggle feature to switch between passthrough and no passthrough so it has the full demonstration of what is discussed in the manual.

@BastiaanOlij BastiaanOlij force-pushed the openxr-passthrough-demo branch from 4a37a1c to d9f27d0 Compare November 5, 2025 01:29

@AThousandShips AThousandShips left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style and writing looks good now!

Comment thread xr/openxr-passthrough-demo/main.gd Outdated
@BastiaanOlij BastiaanOlij force-pushed the openxr-passthrough-demo branch from d9f27d0 to 90236f4 Compare November 10, 2025 09:51
Comment thread xr/openxr-passthrough-demo/README.md Outdated
@BastiaanOlij BastiaanOlij force-pushed the openxr-passthrough-demo branch from 90236f4 to 50c8f13 Compare November 20, 2025 02:51
elif XRInterface.XR_ENV_BLEND_MODE_ADDITIVE in modes:
xr_interface.environment_blend_mode = XRInterface.XR_ENV_BLEND_MODE_ADDITIVE
else:
push_error("Passthrough not supported!")

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like having some feedback inside the demo itself would also be nice to have if passthrough isn't supported by the runtime.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added a message that fades out after time that presents this information to the user if applicable.

You can test it on a Meta Quest by simply turning passthrough support of in project settings.

Its maybe a little overkill but its a fun solution :)

@BastiaanOlij BastiaanOlij force-pushed the openxr-passthrough-demo branch from 50c8f13 to 270f717 Compare November 26, 2025 08:54
@Kimau

Kimau commented Dec 5, 2025

Copy link
Copy Markdown

This looks good from my review during the morning meeting.

@Calinou Calinou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@Calinou

Calinou commented May 4, 2026

Copy link
Copy Markdown
Member

@BastiaanOlij Before merging, can you rename the openxr-passthrough-demo folder to openxr_passthrough? We use snake_case for all demo folder names, and having _demo in a folder name is redundant.

@BastiaanOlij BastiaanOlij force-pushed the openxr-passthrough-demo branch from 270f717 to e92e498 Compare May 5, 2026 03:40
@BastiaanOlij

Copy link
Copy Markdown
Contributor Author

@Calinou no worries, folder has been renamed :)

@Calinou Calinou merged commit f657234 into godotengine:master May 5, 2026
2 checks passed
@BastiaanOlij BastiaanOlij deleted the openxr-passthrough-demo branch May 6, 2026 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants