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

Matrix properties incorrectly declared outside cbuffer during batch save #375

Open
iris-futur opened this issue Jan 27, 2025 · 2 comments
Open
Assignees
Labels
Bug Something isn't working
Milestone

Comments

@iris-futur
Copy link

We ran into an issue where, for a URP shader with a Matrix4x4 property, the property was not included in the cbuffer as it should be. The issue is easiest to reproduce consistently when using the Batch Save and Pack window.

Image

Image

Image

In trying to recreate a minimal example, I found that the bug depends on the value of internal node IDs. I attach two versions of a trivial shader, one which reproduces the issue and one which does not.

BadCBufferRepro.zip

Running with a debugger attached, in Matrix4X4Node.GenerateShaderForOutput, CurrentParameterType is Global instead of Property. I have not fully investigated the logic, but we have worked around the issue locally by hard-coding ParentGraph.IsSRP to always return true.

@Dawie3565
Copy link
Collaborator

@iris-futur
thank you for the report

I see your ASE version you're in is v1.9.6.3 dated around 8/15/2014

Many ASE matrix nodes have since updated
please update to the latest ASE version and try again

Thank you
David

@iris-futur
Copy link
Author

Hi David, thanks for pointing that out, I've now updated to the latest 1.9.8.1. Unfortunately the same issue occurs. I have attached an updated version of the shaders saved with 1.9.8.1.

BadCBufferRepro-1.9.8.1.zip

@diogovtx diogovtx added this to the v1.9.8.2 milestone Jan 28, 2025
@diogovtx diogovtx self-assigned this Jan 28, 2025
@diogovtx diogovtx added the Bug Something isn't working label Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants