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

Fix texture copy for Forward+ renderer #41

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

patrickdown
Copy link
Collaborator

Moved the texture copy from XRInterfaceExtension's the _end_frame to _post_draw_viewport. This seems to fix the fix the barrier issue with T5's rendering. Output seems smooth now with no glitches.

Moved the texture copy from XRInterfaceExtension's the _end_frame to _post_draw_viewport. This seems to fix the fix the barrier issue with T5's rendering. Output seems smooth now with no glitches.
@patrickdown patrickdown self-assigned this Oct 9, 2023
@@ -13,6 +13,7 @@ config_version=5
config/name="TiltFiveGodot4"
run/main_scene="res://main.tscn"
config/features=PackedStringArray("4.1")
run/max_fps=60
Copy link
Member

Choose a reason for hiding this comment

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

Did you mean to add this in?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not really but it shouldn't do any harm. I've been told the T5 won't run over 60fps.

Copy link
Member

@BastiaanOlij BastiaanOlij left a comment

Choose a reason for hiding this comment

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

lgtm, that should do as a solution until we can properly submit the original render texture with layers.

@patrickdown patrickdown merged commit 2a64a7e into GodotVR:main Oct 10, 2023
2 checks passed
@patrickdown patrickdown deleted the fix_texture_copy_output branch November 14, 2023 17:11
@BastiaanOlij BastiaanOlij added the bug Something isn't working label Nov 17, 2023
@BastiaanOlij BastiaanOlij added this to the 1.0.0 milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants