ToonShader did not receive flat and gradient Ambient

[Thaina]

Describe the bug
ToonShader did not receive flat and gradient Ambient

I don't see toonshader use any unity_AmbientEquator unity_AmbientGround unity_AmbientSky to set light source color

To Reproduce
Steps to reproduce the behavior:

1. Using any model with ToonShader
2. Click on Window > Rendering > Lighting > Environment
3. Change Environment Lighting source to Gradient and tweak any color
4. Other model with UniversalShader/Lit change color from ambient but toonshader would not

Expected behavior
toonshader would colored from all ambient gradient as well as other lit shader

Screenshots

Normal

Current

Expect

Desktop (please complete the following information):

• Windows
• Chrome

Additional context
Also I don't understand how unity's own Lit shader can get AmbientMode. I would like to fix it myself and made pull request but this AmbientMode have me confused

[sindharta-tanuwijaya]

Hello.
Thank you very much for your feedback and even a PR to fix this issue !

Just in case, would it be possible for you to share this test project that you are using ?

In the meantime, I'd like to let you know that due to the company reset last year, we at Unity have had to deprioritize the development of the ToonShader package at the moment.
Consequently, we have very limited resources at the moment to address existing bugs, although the above info that you provide will be beneficial for discussions with the community.

That being said, this could change in the near future, though I can’t provide any guarantees at this time.

We appreciate your understanding and support.

[Thaina]

For test project, I think you could use the test scene in this repo directly

ToonShader scene would be easiest one I think

Setting the material's Light Probe Intensity to 0.5-1 and change Environment lighting source to Color or Gradient to test

[Thaina]

@sindharta-tanuwijaya To be honest I am not really sure if my fix was the right formula. It just seem alright but maybe it should not related to env / intensity and GI, or it is?

[sindharta-tanuwijaya]

Thank you for sharing your honest opinion! :)
I just wanted to run the CI tests first to identify any errors that might come up.

And yes, before merging the fix into the main branch, we'll make sure to thoroughly validate it.

[H3idi-X]

@Thaina

Hi, thank you for reporting the issue.
I tried to reproduce the problem, but as far as I can see, the Unity-Chan face appears to be affected by Environmental Lighting. Could you provide more details about the issue, or share a reproducible project with us?

2025-03-27.08-46-33.mp4

[Thaina]

@H3idi-X I am using test project from this repo directly

https://github.com/Unity-Technologies/com.unity.toonshader/tree/development/v1/TestProjects/UniversalGraphicsTest-6000.0

The commit is 1ded2ba

And this is what I see

Or are there any setting I need to set before the material will take ambient light?

[H3idi-X]

Thanks for providing more detailed information.
We succeeded to produce the isuue.
We will check what cause the difference between what I posted yesterday and the following.

2025-03-28.12-20-27.mp4

[H3idi-X]

Hi, we have confirmed that this issue occurs in versions newer than 6000.0. We are investigating the most appropriate fix to maintain compatibility.

[H3idi-X]

Hi, @Thaina
It looks like we’ve found the root cause of this issue. As of Unity 60000.0.12f1, the initialization process in URP lit shaders is now separated, as shown here:
https://github.com/Unity-Technologies/Graphics/blame/master/Packages/com.unity.render-pipelines.universal/Shaders/BakedLitForwardPass.hlsl#L171

Because of this change, we'll need to add code to initialize GI.
We're currently testing a fix in our CI pipeline, which you can find here:
https://github.com/Unity-Technologies/com.unity.toonshader/pull/414/files

We'll let you know once we've confirmed it resolves the problem.

[H3idi-X]

We have confirmed that this issue has been resolved with the following fix:
#414

Therefore, we are closing this issue. If you have any further problems, please feel free to reopen it.