Search Issue Tracker
Fixed in 2021.2.X
Fixed in 2021.1.X
Votes
0
Found in
2020.1.0a22
Issue ID
1323393
Regression
Yes
Performance regression when baking light probes with a light cookie in the scene
Introduced by https://ono.unity3d.com/unity/unity/pull-request/99050/_/lighting/plm/lightcookies
OpenCLCommonBuffers::PrepareLightCookiesBuffer is called 500+ times during the bake, because SetSceneStateOnTechnique(this) is called from RadeonRaysBakeLightProbesTechnique::Render()!
This should only be done once, in RadeonRaysBakeLightProbesTechnique::AttachHeavyDataPointers. SetSceneStateOnTechnique isn't tread safe so this can also cause crashes if scene state is changed while baking.
Performance hit was visible in https://confluence.unity3d.com/pages/viewpage.action?spaceKey=teamgi&title=Light+cookie+support+for+PLM but ignored. The more light probe groups and the higher sample counts you have, the worse it gets.
https://ono.unity3d.com/unity/unity/pull-request/99050/_/lighting/plm/lightcookies landed in trunk at 2020.1.0a22.2693_a851a673d667
All about bugs
View bugs we have successfully reproduced, and vote for the bugs you want to see fixed most urgently.
Latest issues
- [Android][BiRP] Depth processing is handled incorrectly on certain Android devices when using 2 camera's
- [APV] Cancelling Display Dialog Error is thrown after Adaptive Probe Volumes tab is open in Lighting Window
- [APV] NullReferenceException is thrown when baking Adaptive Probe Volume for a Terrain with Non-GI Contributing Tree Prototypes and multiple APV objects with different LayerMasks are present on the scene
- Hands are not recognized when using Hololens 2
- "OnTriggerExit2D" is called before "OnTriggerEnter2D" when object is destroyed immediately
Add comment