Skip to content

Conversation

adrien-de-tocqueville
Copy link
Contributor

@adrien-de-tocqueville adrien-de-tocqueville commented Sep 10, 2020

Fix for https://fogbugz.unity3d.com/f/cases/1211957/
and https://fogbugz.unity3d.com/f/cases/1264734/

HDRP overrides the lod settings from QualitySettings just when rendering, then put back to their initial values.
So when the LODGroup component inspector is draw, it uses some meaningless values.

But for rendering, the lod settings can be changed by the framesettings, so I use the one from the HDRP default frame settings, which are the one used by the editor camera.

I tested that the value displayed in the LODGroup inspector is correct
and that the LOD transitions in the viewport match the UI

to have consistent behavior with the camera editor
@adrien-de-tocqueville adrien-de-tocqueville changed the title Override bias value with the one from HDRP in low quality Override LOD settings with the one from HDRP Sep 10, 2020
@github-actions
Copy link

It appears that you made a non-draft PR!
Please convert your PR to draft (button on the right side of the page)
and cancel any jobs that started on Yamato.
See the PR template for more information.
Thank you!

@JulienIgnace-Unity JulienIgnace-Unity requested a review from a team September 11, 2020 08:02
Copy link
Contributor

@JMargevics JMargevics left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I0vtvVBpaP

Checked in both HDRP Default settings and camera Custom Frame Settings, Inspector warning message and Scene UI now show correct LOD Bias values ✅

@JulienIgnace-Unity
Copy link
Contributor

The fix is included in a bigger LoD related PR: #2072

@adrien-de-tocqueville adrien-de-tocqueville deleted the HDRP/use-hdrp-lod-bias branch October 6, 2020 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants