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

MToon10のURP向けシェーダの不具合やパス不備の修正 #2110

Merged
merged 5 commits into from
Jul 26, 2023

Conversation

yoship1639
Copy link
Contributor

MToon10のURP向けシェーダに不具合、Passの不備があったため以下の修正を行いました。
ご確認の程よろしくお願いいたします。

  • ShadowCasterの不具合修正

Cutoutモデルのキャストシャドーが正常に行われていない不具合を修正しました。
これは、AlphaClipを行うべきUV座標が正しくFragmentシェーダに渡されていなかったため発生していました。

  • 不要なBuilt-inキーワードの削除

URPシェーダにおいて必要のないBuilt-in専用のキーワード群を削除しました。
以下はBuilt-inシェーダ専用なので不要です。
#pragma multi_compile_fwdbase nolightmap nodynlightmap nodirlightmap novertexlight

  • DepthOnlyPass、DepthNormalsPassの追加

DepthOnlyとDepthNormalsパスを追加しました。
_CameraDepthTextureや_CameraNormalsTextureを使うポストプロセスには上記のパスはほぼ必須であり、また、DepthPrepassが使われるプロジェクトはDepthOnlyもしくはDepthNormalsパスがPrepassとして使われるため、URPにおいてはこれらが存在しないと様々なポストプロセス表現や正常な深度テストが行えない場合があります。

@vrm-github-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@ousttrue ousttrue requested review from notargs and Santarh July 26, 2023 03:47
@ousttrue ousttrue added this to the v0.113 milestone Jul 26, 2023
Copy link
Contributor

@notargs notargs left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Santarh
Copy link
Contributor

Santarh commented Jul 26, 2023

Jenkins

@ousttrue ousttrue merged commit d988333 into vrm-c:master Jul 26, 2023
1 check passed
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.

None yet

5 participants