Skip to content

Change RenderPipelineUtility to use GraphicsSettings.currentRenderPipeline#2741

Merged
Santarh merged 1 commit intovrm-c:masterfrom
redwyre:patch-1
Apr 10, 2026
Merged

Change RenderPipelineUtility to use GraphicsSettings.currentRenderPipeline#2741
Santarh merged 1 commit intovrm-c:masterfrom
redwyre:patch-1

Conversation

@redwyre
Copy link
Copy Markdown
Contributor

@redwyre redwyre commented Oct 26, 2025

#2740

Changes the use of RenderPipelineManager.currentPipeline in RenderPipelineUtility.GetRenderPipelineType() which isn't valid for the first few frames, to GraphicsSettings.currentRenderPipeline, which is.

@redwyre
Copy link
Copy Markdown
Contributor Author

redwyre commented Apr 7, 2026

@ousttrue @Santarh Sorry to ping, but can someone review so I don't have to keep rebasing? :P

@Santarh
Copy link
Copy Markdown
Contributor

Santarh commented Apr 10, 2026

Sorry for the long delay on this!
I've been occupied with other commitments and wasn't able to give this the attention it deserved.
Thank you so much for your patience — the changes look good, merging now.

@Santarh Santarh merged commit 1adda69 into vrm-c:master Apr 10, 2026
1 check passed
@redwyre
Copy link
Copy Markdown
Contributor Author

redwyre commented Apr 10, 2026

Thanks, I appreciate you taking the time the review and merge :)

@redwyre redwyre deleted the patch-1 branch April 10, 2026 05:08
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.

3 participants