Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HDRP Template: new material sphere and various art fixes #7114

Merged
merged 10 commits into from
Feb 18, 2022

Conversation

pierre-unity
Copy link
Contributor

@pierre-unity pierre-unity commented Feb 11, 2022

Purpose of this PR

  • Added new material sphere in Room 1 and 3, removed previous one, and updated cinematic as a result
    image
    image

  • Removed sRGB flag on all detail maps, and tuned corresponding materials to compensate for visual change (very minor change)
    image

  • Tuned shadows (cascades, contact shadows, directional light biases) to maximize quality on the new material sphere, at both Medium and High quality
    image

  • Fixed bad normals on metal trims in room 3
    image


Comments to reviewers

Very minor changes, mostly artistic related.

@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).
See the PR template for more information.
Thank you!

@pierre-unity pierre-unity marked this pull request as draft February 11, 2022 13:55
@pierre-unity pierre-unity marked this pull request as ready for review February 11, 2022 13:59
Copy link
Contributor

@sebastienlagarde sebastienlagarde left a comment

Choose a reason for hiding this comment

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

Note for myself: this PR affect default lookdev profile and default volume

iM0ve
iM0ve previously requested changes Feb 14, 2022
Copy link
Contributor

@iM0ve iM0ve left a comment

Choose a reason for hiding this comment

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

Changing the default volume breaks the tutorial(can no longer follow instructions). We should either revert the default volume or update the tutorial.

Screen.Recording.2022-02-14.at.12.51.11.mov

@sebastienlagarde
Copy link
Contributor

@pierre-unity please could you separate the change on the resources inside the hdrp package from the change of the template in differente PR? This week we are proceeeding to the move to private repo and your PR could be problematic, need to split it.

@pierre-unity
Copy link
Contributor Author

pierre-unity commented Feb 15, 2022

Thanks, I'll do this today ASAP. @sebastienlagarde

Very odd these changes in the global resources btw (volume profiles), I thought it was part of the upgrade process, that's why I pushed them (I only inspected these files, they must have changed automatically).

I'll also update the tutorial to reflect the changes to the default global volume. Thanks @iM0ve.

@pierre-unity
Copy link
Contributor Author

I've reverted the change to non-template files.

I've also reverted the change to the global profile, and therefore the tutorial is working as expected again.

Ready to merge.

@pierre-unity pierre-unity dismissed iM0ve’s stale review February 16, 2022 11:44

Thank you, the change to the global volume profile has been reverted.

@sebastienlagarde
Copy link
Contributor

@joeksy / @iM0ve template updated, can you check, thanks

Copy link
Contributor

@iM0ve iM0ve left a comment

Choose a reason for hiding this comment

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

The RP can land. I have found some issues that we might want to address later regarding the template but they are not introduced by this change.

What's tested:

  • Scene visual with the new changes
  • Some bits of tutorial
  • Entering playmode
  • Launching cinematics
  • Creating new scenes

Issues:

  1. Entering playmode results in "missing scripts" warnings
    Unity_EAIcxZcsiU

  2. Creating new indoor scene results in an overly bright image. Caused by volumetric clouds.
    m5UaiyWUAA

  3. Small suggestion to make it easier to use cinematics. Both parent object (Media) and child object (Cinematic timeline) are disabled. Users need to check 2 checboxes to see the cinematic, I would suggest to enable the child and keep partent disabled so its easier to access. TLDR check the checbox in the image

image

@sebastienlagarde sebastienlagarde merged commit 6022b50 into master Feb 18, 2022
@sebastienlagarde sebastienlagarde deleted the hdrp-template-new-material-sphere-and-fixes branch February 18, 2022 11:07
sebastienlagarde pushed a commit that referenced this pull request Apr 8, 2022
* added new material sphere, tuned contact shadows, and overall shadowing quality

* recaptured all reflection probes, removed srgb on detail maps, tuned shadows, updated cutscene

* medium quality by default

* slightly tuned exposure transition in corridor to room3

* fixed bad normal on metal trims

* reverted changes to default volume profiles

* reassigned default volume profile to level to avoid issues with tutorial

* swapped new material ball for the new one in core resources to avoid duplicate
@iM0ve iM0ve restored the hdrp-template-new-material-sphere-and-fixes branch May 16, 2022 13:14
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