Skip to content

Commit e10f030

Browse files
Fixed an issue where forced sky update (like PBR sky amortized updated) would not update ambient probe. (#6831)
Co-authored-by: sebastienlagarde <[email protected]>
1 parent 04c6451 commit e10f030

File tree

3 files changed

+2
-4
lines changed

3 files changed

+2
-4
lines changed

com.unity.render-pipelines.high-definition/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
7979
- Fixed a null ref exception in Volume Explorer
8080
- Fixed one frame flicker on hardware DRS - (case 1398085)
8181
- Fixed using the wrong coordinate to compute the sampling direction for the screen space global illumination.
82+
- Fixed an issue where forced sky update (like PBR sky amortized updated) would not update ambient probe.
8283

8384
## [14.0.0] - 2021-11-17
8485

com.unity.render-pipelines.high-definition/Runtime/Sky/SkyManager.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1066,12 +1066,11 @@ void UpdateEnvironment(
10661066
{
10671067
var skyCubemap = GenerateSkyCubemap(renderGraph, hdCamera, skyContext);
10681068

1069-
if (updateAmbientProbe && !renderingContext.computeAmbientProbeRequested)
1069+
if (updateAmbientProbe)
10701070
{
10711071
UpdateAmbientProbe(renderGraph, skyCubemap, outputForClouds: false,
10721072
renderingContext.ambientProbeResult, renderingContext.diffuseAmbientProbeBuffer, renderingContext.volumetricAmbientProbeBuffer,
10731073
hdCamera.volumeStack.GetComponent<Fog>(), renderingContext.OnComputeAmbientProbeDone);
1074-
renderingContext.computeAmbientProbeRequested = true;
10751074
}
10761075

10771076
if (renderingContext.supportsConvolution)

com.unity.render-pipelines.high-definition/Runtime/Sky/SkyRenderingContext.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ internal class SkyRenderingContext
1717
public bool supportsConvolution { get; private set; } = false;
1818

1919
internal bool ambientProbeIsReady = false;
20-
public bool computeAmbientProbeRequested = false;
2120

2221
public SkyRenderingContext(int resolution, int bsdfCount, bool supportsConvolution, SphericalHarmonicsL2 ambientProbe, string name)
2322
{
@@ -51,7 +50,6 @@ public SkyRenderingContext(int resolution, int bsdfCount, bool supportsConvoluti
5150
public void Reset()
5251
{
5352
ambientProbeIsReady = false;
54-
computeAmbientProbeRequested = false;
5553
}
5654

5755
public void Cleanup()

0 commit comments

Comments
 (0)