Skip to content

Initial XR support#3245

Draft
AoGao-Kedoka wants to merge 8 commits into
f3d-app:masterfrom
AoGao-Kedoka:xr
Draft

Initial XR support#3245
AoGao-Kedoka wants to merge 8 commits into
f3d-app:masterfrom
AoGao-Kedoka:xr

Conversation

@AoGao-Kedoka

@AoGao-Kedoka AoGao-Kedoka commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

Adding XR support in rendering. Interacting is handled in another PR.
scale and location of the object in the scene still needs to be handled.

Issue ticket number and link if any

#749

Checklist for finalizing the PR

  • I have performed a self-review of my code
  • I have added tests for new features and bugfixes
  • I have added documentation for new features
  • If it is a modifying the libf3d API, I have updated bindings
  • If it is a modifying the .github/workflows/versions.json, I have updated docker_timestamp

AI Disclosure

  • I did not use AI to generate any of the content of that pull request
  • I used AI to generate code in that pull request, if yes please disclose which part of the code was generated and with which model.
    GPT5.5 for Creating the bounding box and align objects to the bounding box. Reviewed and tested carefully.
  • ...

Continuous integration

Please write a comment to run CI, eg: \ci fast.
See here for more info.

@AoGao-Kedoka AoGao-Kedoka requested a review from a team as a code owner June 16, 2026 19:17
@AoGao-Kedoka AoGao-Kedoka marked this pull request as draft June 16, 2026 19:18
@github-actions

Copy link
Copy Markdown

You are modifying libf3d public API! ⚠️Please update bindings accordingly⚠️!
You can find them in their respective directories: c, python, java, webassembly.

@mwestphal mwestphal 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.

Nice!

overlayP->SetDelegatePass(renderingPass);

this->SetPass(overlayP);
this->SetPass(overlayP);

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.

Don't you still need to call this->SetPass(renderingPass); when XR is enabled in order to have all other features (AA, AO, blending...)?

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.

yeah, that's also the next step i need to validate. There's still graphics bugs with vtkF3DRenderpass that i need to fix.
Will keep this one unresolved until it's being fixed.

Comment thread library/CMakeLists.txt Outdated
Comment thread library/src/interactor_impl.cxx
@AoGao-Kedoka AoGao-Kedoka force-pushed the xr branch 3 times, most recently from 992389b to 9796670 Compare June 26, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants