Search Unity

  1. Welcome to the Unity Forums! Please take the time to read our Code of Conduct to familiarize yourself with the forum rules and how to post constructively.
  2. Unite 2023 Registrations are now LIVE! We are thrilled to announce that Unite 2023 Registration is now live and open for all!
    Dismiss Notice
New Forum User Notice Unite 2023 Registrations are now LIVE!

Bug IN-30145 - Probe volume reflection probe normalization is always invalid 2022.2

Discussion in 'High Definition Render Pipeline' started by PutridEx, Jan 27, 2023.

  1. PutridEx

    PutridEx

    Joined:
    Feb 3, 2021
    Posts:
    1,120
    Doesn't work anymore in 2022.2, IN-30145
     
    PutridEx, Jan 27, 2023
    #1

  2. francescoc_unity

    francescoc_unity

    Unity Technologies

    Joined:
    Sep 19, 2018
    Posts:
    193
    We recently got a "fix" for this, but not backported, though if you are using a custom branch it is trivial to add it on your end if you want.

    upload_2023-1-27_10-12-45.png


    Just to give a bit of context, the refl probe normalization most of the time actually does the right thing, but is very easy to hit at last a single back face; when that happens with previous code the refl probe would immediately think it is completely in an invalid position and will discard the data.


    We'll probably backport the fix as we process your back, but I included the snippet above just in case is needed before
     
    francescoc_unity, Jan 27, 2023
    #2
    ftejada and PutridEx like this.

  3. Ryan-Hayle

    Ryan-Hayle

    Joined:
    Feb 16, 2014
    Posts:
    142
    "Baked reflection probe normalization data is invalid." - Why do you get this warning on reflection probes on a basic scene with AVP enabled? Is this the issue you are referring to? I see that it goes away after baking in Unity 2023.1, but not in Unity 2022.2.15
     
    Last edited: Apr 15, 2023
    Ryan-Hayle, Apr 15, 2023
    #3

  4. SebLagarde

    SebLagarde

    Unity Technologies

    Joined:
    Dec 30, 2015
    Posts:
    932
    Hi, do you have repro step for this issue as we don't reproduce it ? (and on which version), thanks
     
    SebLagarde, Apr 17, 2023
    #4

  5. Ryan-Hayle

    Ryan-Hayle

    Joined:
    Feb 16, 2014
    Posts:
    142
    I'm using Unity 2022.2.15 HDRP and it just remains present all the time.

    Here is an example project:
    https://1drv.ms/u/s!An99pWyNHVKijgckikSU80vfI4yL?e=Eacats
     
    Ryan-Hayle, Apr 17, 2023
    #5

  6. Ryan-Hayle

    Ryan-Hayle

    Joined:
    Feb 16, 2014
    Posts:
    142
    upload_2023-4-17_10-26-2.png
     
    Ryan-Hayle, Apr 17, 2023
    #6
    JamesArndt likes this.

  7. JamesArndt

    JamesArndt

    Unity Technologies

    Joined:
    Dec 1, 2009
    Posts:
    2,912
    Just caught this issue working on my solo dev project. Latest Unity 2022.3.0 LTS release using HDRP 14.0.7. Same error on my reflection probe when I bake with Adaptive Probe Volumes enabled. Interestingly enough I see the code fix (seen above) when I checked ProbeReferenceVolume.ReflProbeNormalization.cs

     
    JamesArndt, Jun 4, 2023
    #7

  8. PutridEx

    PutridEx

    Joined:
    Feb 3, 2021
    Posts:
    1,120
    Yeah, seems to still happen. I remember testing it by adding the above code manually a while ago and the issue persisted.
     
    PutridEx, Jun 4, 2023
    #8
    JamesArndt likes this.

  9. Skokon

    Skokon

    Joined:
    Jul 23, 2017
    Posts:
    74
    I'm using unity 2023.2.0a17

    When i bake the probes from baking settings it says the data is valid

    but when i move any reflection probe the reflection changes and it says the data is invalid and if i create new one it says the data is invalid
     
    Skokon, Jun 10, 2023
    #9

  10. RevC2

    RevC2

    Joined:
    May 2, 2021
    Posts:
    48
    Bug it's still present in 2022.3.2.LTS, will the fix be backported?
     
    RevC2, Jun 22, 2023
    #10

  11. adrien-de-tocqueville

    adrien-de-tocqueville

    Unity Technologies

    Joined:
    Mar 16, 2020
    Posts:
    163
    This is expected, if you move the probe data needs to be rebaked (from the lighting tab, simply baking the reflection probe will not fix it)
     
    adrien-de-tocqueville, Jun 22, 2023
    #11

  12. adrien-de-tocqueville

    adrien-de-tocqueville

    Unity Technologies

    Joined:
    Mar 16, 2020
    Posts:
    163
    The fix has been backport on 22.3, maybe you are seing something else
     
    adrien-de-tocqueville, Jun 22, 2023
    #12

  13. RevC2

    RevC2

    Joined:
    May 2, 2021
    Posts:
    48
    Strange, since I was having this problem I tried my scene in 2023 and reflection probes normalization do work. In 2022 LTS the still say "normalization data is invalid"
     
    RevC2, Jun 22, 2023
    #13
    ebaender likes this.

  14. ebaender

    ebaender

    Joined:
    Oct 29, 2020
    Posts:
    97
    Same for me, it's just broken in 2022.3.
     
    ebaender, Jun 22, 2023
    #14

  15. JamesArndt

    JamesArndt

    Unity Technologies

    Joined:
    Dec 1, 2009
    Posts:
    2,912
    JamesArndt, Jun 22, 2023
    #15
    kmowers and Gasimo like this.

  16. adrien-de-tocqueville

    adrien-de-tocqueville

    Unity Technologies

    Joined:
    Mar 16, 2020
    Posts:
    163
    adrien-de-tocqueville, Jun 22, 2023
    #16
    JamesArndt likes this.