Skip to content

Conversation

@Elfi0Kuhndorf
Copy link
Contributor

Description

Cal Chiu helped our team and adjusted settings and shader to fix the lighting and shadow display within the project. Further with the help of MingWai we fixed the transparency issue of the water. This PR helps to improve the overall look of the sample.

Issue Number(s)

https://jira.unity3d.com/browse/MTT-9279

Contribution checklist

  • Tests have been added for the project and/or any internal package
  • Release notes have been added to the project changelog file
  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • JIRA ticket ID is in the PR title or at least one commit message
  • Include the ticket ID number within the body message of the PR to create a hyperlink

CalChiu and others added 5 commits November 14, 2024 14:18
- Rebaked lightmaps
- Commented some keywords in Assets/Shaders/CustomHLSL/CustomLighting.hlsl for shadows
- Added _MAIN_LIGHT_SHADOWS keyword to Assets/Shaders/SubGraphs/ToonShading.shadersubgraph
- Switched keywords to "Multi Compile" instead of "Shader Feature" in Assets/Shaders/SubGraphs/ToonLightingModel.shadersubgraph
- Switched the directional light mode to Mixed from Baked in the TownMarket scene
- Enabled shadow cascade keyword
- Edited Assets/Shaders/SubGraphs/ToonLightingModel.shadersubgraph to force enable light map multiplier
- Set main light cascade settings to match original project
…enabling opaque and depth texture settings in RPAssets
@unity-cla-assistant
Copy link

unity-cla-assistant commented Dec 4, 2024

CLA assistant check
All committers have signed the CLA.

@Elfi0Kuhndorf Elfi0Kuhndorf marked this pull request as ready for review December 4, 2024 13:07
@Elfi0Kuhndorf Elfi0Kuhndorf requested a review from a team as a code owner December 4, 2024 13:07
@hammerlerobi hammerlerobi self-assigned this Dec 6, 2024
hammerlerobi
hammerlerobi previously approved these changes Dec 9, 2024
Copy link
Contributor

@hammerlerobi hammerlerobi left a comment

Choose a reason for hiding this comment

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

Tested on Mobile and MacOS, works and great improvement.

  • Maybe we should remove the uncommented sections from CustomLighting.hlsl? (Can be done later)
  • Investigate Shadow Poping in a follow up.

Ready to merge

Copy link
Collaborator

@fernando-cortez fernando-cortez left a comment

Choose a reason for hiding this comment

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

Looks absolutely amazing!
Let's add a note in the changelog before merging 😄

Copy link
Contributor

@hammerlerobi hammerlerobi left a comment

Choose a reason for hiding this comment

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

Great improvements

@Elfi0Kuhndorf Elfi0Kuhndorf merged commit 190f3c2 into develop Dec 16, 2024
27 of 29 checks passed
@Elfi0Kuhndorf Elfi0Kuhndorf deleted the fix/distributed-authority-sample/cal-lighting-shadow-and-water-fix branch December 16, 2024 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants