Skip to content

[HDRP][Path Tracing] Backport of sky+recorder fix #5001

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

Merged
merged 2 commits into from
Jul 12, 2021

Conversation

eturquin
Copy link
Contributor

Backport of #4856

Addresses: https://fogbugz.unity3d.com/f/cases/1340507

Before: When recording (using the recorder package) in path tracing mode, sky settings would be ignored (sky was always set to clear color).
Now: Settings are properly taken into account.

This has been tested on the HDRP template (which was the scene used as an example in the accompanying ticket).

@github-actions
Copy link

It appears that you made a non-draft PR!
Please convert your PR to draft (button on the right side of the page)
and cancel any jobs that started on Yamato.
See the PR template for more information.
Thank you!

@sebastienlagarde sebastienlagarde merged commit bea104b into 2021.1/staging Jul 12, 2021
@sebastienlagarde sebastienlagarde deleted the hd/pt_record_sky_backport branch July 12, 2021 13:56
sebastienlagarde added a commit that referenced this pull request Aug 6, 2021
* Improved a bit the area cookie behavior for higher smoothness values to reduce artifacts. #3578

* [SRP] Add Conditional field for shader code generation in Generate include command #4889

* Fix VT Feedback buffer #4868

* [HDRP][Path Tracing] Backport of sky+recorder fix #5001

* [HDRP] Fix distortion when resizing the graphics compositor window in builds #5062

* Add compatibility with Unity 2020.3 for compositor resize callback

* Small cleanup

Co-authored-by: anisunity <42026998+anisunity@users.noreply.github.com>
Co-authored-by: Yao Xiaoling <yao@unity3d.com>
Co-authored-by: Emmanuel Turquin <emmanuel@turquin.org>
Co-authored-by: Pavlos Mavridis <pavlos.mavridis@unity3d.com>
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.

2 participants