[HDRP] Disable volumetric clouds in lens flare sample indoor scene by remi-chapelain · Pull Request #6953 · Unity-Technologies/Graphics · GitHub
Skip to content

[HDRP] Disable volumetric clouds in lens flare sample indoor scene #6953

Merged
sebastienlagarde merged 3 commits into
masterfrom
HDRP/fix-indoor-lensflare-sample
Feb 2, 2022
Merged

[HDRP] Disable volumetric clouds in lens flare sample indoor scene #6953
sebastienlagarde merged 3 commits into
masterfrom
HDRP/fix-indoor-lensflare-sample

Conversation

@remi-chapelain

@remi-chapelain remi-chapelain commented Feb 2, 2022

Copy link
Copy Markdown
Contributor

Purpose of this PR

Following this PR, the same fix is needed for the lens flare indoor sample scene.


Testing status

Switched between scenes (outdoor, indoor) with and without always refresh. ✔️


Comments to reviewers

Needs backport 2021.2 and need to check after backport that the volume has the proper override since we had issue with backport for the indoor template scenes

@github-actions

github-actions Bot commented Feb 2, 2022

Copy link
Copy Markdown

@github-actions github-actions Bot added the HDRP label Feb 2, 2022
@sebastienlagarde sebastienlagarde merged commit d6c0c8c into master Feb 2, 2022
@sebastienlagarde sebastienlagarde deleted the HDRP/fix-indoor-lensflare-sample branch February 2, 2022 11:12
sebastienlagarde added a commit that referenced this pull request Feb 2, 2022
* Changed the roughness value used for screen space reflections and ray traced reflections to match environment lighting (case 1390916). #680

* [HDRP] Improve custom pass name UI #6859

* Fixed using the wrong directional light data for clouds and the definition of current Sun when the shadow pass is culled (case 1399000) #6925

* [HDRP] Disable volumetric clouds in lens flare sample indoor scene #6953

* [Hdrp] Merge Hd/bugfix #6954

* Update HDAdditionalLightData.cs

Co-authored-by: anisunity <42026998+anisunity@users.noreply.github.com>
Co-authored-by: Antoine Lelievre <antoinel@unity3d.com>
Co-authored-by: Rémi Chapelain <57442369+remi-chapelain@users.noreply.github.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