-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add a variable rate shading demo #852
Add a variable rate shading demo #852
Conversation
c5413d9
to
57f3e70
Compare
Will be testing this today. |
get_viewport().use_xr = true | ||
xr_camera.current = true | ||
|
||
# Reposition our origin point to work around an engine bug. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this bug still active?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I commented out the line and the bug occurred in 4.0.stable, so yes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, weird, I was under the impression that was fixed. I might need to have another look at that :)
texture_rect.texture = texture | ||
texture_rect.visible = true | ||
elif vrs_mode == Viewport.VRS_XR: | ||
# Doesn't seem to be supported yet. This should be exposed in a future engine version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still a problem?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To my knowledge, yes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is just a matter of binding get_vrs_texture
in XRInterface
so we have access to it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there is room for improvement but for a first version of a demo that will give people some instruction on how to use this, it's good enough.
This project showcases how to use variable rate shading (VRS) in 3D to improve performance. Performance metrics are also displayed to evaluate potential performance gains.
57f3e70
to
fc65fd6
Compare
This project showcases how to use variable rate shading (VRS) in 3D to improve performance.
Performance metrics are also displayed to evaluate potential performance gains.
Once this PR is merged, we can link to the demo in the documentation 🙂
Preview