Skip to content

[2021.2] [ShaderGraph] URP Uber Shader #3705

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

Merged
merged 74 commits into from
Apr 29, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
74 commits
Select commit Hold shift + click to select a range
613a536
Working transparent/opaque switch
Feb 27, 2021
5d4ffac
Working surface type, cull mode, and alpha clip
Feb 27, 2021
aa88cf4
Fix for Unlit alpha test shadows
Mar 1, 2021
c11b947
Workflow Mode switch working on Lit
Mar 1, 2021
9a9f688
Lit AlphaTest working
Mar 1, 2021
ff00c31
Fix for Lit ReceiveShadows checkbox
Mar 1, 2021
10b540b
Draw shadergraph properties in material inspector
Mar 1, 2021
ca653cb
Cleanup, Setting up Target defaults correctly
Mar 1, 2021
d131202
Fix for error spew from inspector on blank graphs
Mar 2, 2021
4afe766
Cast Shadows Toggle
Mar 3, 2021
954ab83
Adding LinearGrey and Red default texture options
Mar 3, 2021
24ea748
Adding stage options to Keywords, fixing Keyword control in BaseShade…
Mar 4, 2021
39ea696
Fix for collision with HDRP property names, fix keyworddescriptor wit…
Mar 5, 2021
4190ef4
Fix Previews, Convert target "Two Sided" checkbox to "Render Face" en…
Mar 26, 2021
1a47133
Fix for transparent blending with alpha
Mar 26, 2021
2c94bc6
Missing setup for transparent blending with alpha
Mar 26, 2021
6589a2d
Fix for material alpha clip when shader alpha clip disabled
Mar 26, 2021
31637a2
Make SubTarget UI the same as Material UI, and add cast/receive shado…
Mar 26, 2021
6a5c9e6
Remove debug log spew
Mar 26, 2021
7772681
Add workflow mode material control checkbox, fix default preview prop…
Mar 30, 2021
e126fca
Fixing cull mode in shadow and depth-only passes, removing unused fie…
Mar 30, 2021
8859752
Removing unused PBRMasterGUI, cleaning up code and comments, fixing e…
Apr 1, 2021
aea47f5
Fixing predefined keyword -- should not define anything
Apr 1, 2021
6b7f6bd
Merge branch 'master' into sg/urp-uber
Apr 1, 2021
c489b0c
Keyword test fix
Apr 1, 2021
7a56f74
Add "ShaderGraphShader" tag to all SubTargets. Fix for errors when c…
Apr 2, 2021
1ab31e2
Add versioning to shadergraph materials
Apr 2, 2021
8115bf5
Use UniversalMetadata approach
Apr 2, 2021
0056a1e
Cleanup warning spew
Apr 3, 2021
2aa23b7
Fix for toggles in UI
Apr 3, 2021
9fe609d
Renaming conflicting properties in test shader
Apr 5, 2021
2910fb8
Fixing a warning/error in a test shader in OpenGL (doesn't allow "__"…
Apr 5, 2021
340d10c
Fix for coat mask support when using GL
Apr 5, 2021
e77c2c7
Uniquely named URP surface properties
Apr 7, 2021
53b1b0d
Moving keyword strings to central location
Apr 7, 2021
9578886
Adding Depth Test control, fixing error when selecting Lit.shader
Apr 9, 2021
fb46581
Texture2D properties using Bump default will now be marked with [Norm…
Apr 9, 2021
50114d8
Adding zwrite controls
Apr 9, 2021
17a7867
Unify common target UI
Apr 12, 2021
3f45746
Removing unnecessarily added render pipeline assets
Apr 12, 2021
0a17c4f
Fixing shadergraph material versioning so it only applies to shadergr…
Apr 12, 2021
ae32e7c
Reverting unnecessary changes to test project
Apr 13, 2021
38d5c57
Cleaned up GUI code
Apr 13, 2021
7438249
Removing unused files, cleanup SubTargets
Apr 13, 2021
9fc1398
More target code cleanup
Apr 13, 2021
7103f4c
Adding "Allow Material Override" checkbox. Fixing a few issues with …
Apr 16, 2021
3268688
Switching to Option 3 (and cleanup)
Apr 16, 2021
5ae7a71
Adding auto-update to Materials (temp solution until AssetDB dependen…
Apr 19, 2021
8a5e821
Adding changelogs
Apr 19, 2021
80309ff
Fix for warning
Apr 19, 2021
5be1912
Removing Material v5 Versioning -- not necessary anymore with "Allow …
Apr 19, 2021
bf075b4
Code cleanup pass
Apr 19, 2021
c5869be
More code cleanup, consolidating string declarations and moving code …
Apr 20, 2021
5ef9cbe
Renaming URPLitGui to ShaderGraphLitGUI (and the same for unlit)
Apr 20, 2021
19958f1
Fixup emission properties for ShaderGraph -- don't use hard-coded Lit…
Apr 20, 2021
731d4c9
Cleaning up GI material code, introducing MaterialUpdateType, so we c…
Apr 21, 2021
41581e1
ShaderGraph Materials should always allow direct user control for dou…
Apr 21, 2021
c09117c
Merge branch 'master' into sg/urp-uber
Apr 21, 2021
8ce3904
Fix incorrect conflict merge
Apr 21, 2021
a830886
Adding 050_Shader_Graphs_Override test
Apr 21, 2021
6b6a97f
Hooking up the new test scene
Apr 22, 2021
61e1f44
Removing ZTestMode.Disabled as an option
Apr 22, 2021
4010912
Disabling DepthOnly pass when zwrite is off, removing it from shader …
Apr 22, 2021
2684ca8
Fix syntax error, moving global illumination settings to advanced sec…
Apr 22, 2021
c187236
Moving "Allow Material Override" checkbox above the "workflow mode" s…
Apr 23, 2021
828cfae
Merge branch 'sg/urp-uber-tests' into sg/urp-uber
Apr 23, 2021
5c5f64c
Addressing feedback
Apr 27, 2021
e748dc8
Getting rid of error spew by matching property type to HDRP
Apr 27, 2021
e0d8306
Fixing unreachable code warnings, and fixing colliding keyword (_SURF…
Apr 27, 2021
877125c
Merge branch 'master' into sg/urp-uber
Apr 27, 2021
bcbb7f9
Fixing merge issues
Apr 27, 2021
ce87a2f
Removing automatic RenderQueue calculation behavior from ShaderGraph …
Apr 28, 2021
f4e57f3
Ensuring the GI controls on URP ShaderGraph Materials are identical t…
Apr 28, 2021
ebe155c
Merge branch 'master' into sg/urp-uber
Apr 29, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -195,11 +195,7 @@ public void CanGetKeywordDeclarationForReals()
var sb = new ShaderStringBuilder();
foreach (var keyword in m_Collector.keywords)
{
string declaration = keyword.GetKeywordDeclarationString();
if(!string.IsNullOrEmpty(declaration))
{
sb.AppendLine(declaration);
}
keyword.AppendKeywordDeclarationStrings(sb);
Copy link
Author

Choose a reason for hiding this comment

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

This more optimal form just appends the keyword declaration string directly into the builder... no temp string GC allocs

}

Assert.AreEqual(kExpectedForRealsDeclaration.Replace("\r\n", "\n"), sb.ToString(), "Keyword declaration snippet for final shader was invalid");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,7 @@ public IEnumerator RemoveInputTests()
var menuItems = m_BlackboardTestController.addBlackboardItemsMenu.GetPrivateProperty<IList>("menuItems");
Assert.IsNotNull(menuItems, "Could not retrieve reference to the menu items of the Blackboard Add Items menu");

// invoke all menu items on the "add Blackboard Items Menu" to add all property types
foreach (var item in menuItems)
{
var menuFunction = item.GetNonPrivateField<GenericMenu.MenuFunction>("func");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1124,7 +1124,7 @@
},
"m_Name": "A",
"m_DefaultReferenceName": "Vector1_e29cd19dafd54a1dbd31dcf1d75c21b2",
"m_OverrideReferenceName": "a_3_4_d__",
Copy link
Author

Choose a reason for hiding this comment

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

Fixing an issue with this test on GL devices -- not allowed to use double underscore in identifiers.

"m_OverrideReferenceName": "a_3_4_d_",
"m_GeneratePropertyBlock": true,
"m_Precision": 0,
"m_GPUInstanced": false,
Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading