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

[Editor] Missing field " Name " for Custom Vertex Stream in Material Property Editor #223

Open
jb-perrier opened this issue Sep 27, 2018 · 3 comments
Labels

Comments

@jb-perrier
Copy link

Release Type: Official Release

Version: 3.0.0.5

Platform(s): Windows

Describe the bug
When you select Custom Vertex Stream in the material property grid, there is no field " Name " to select the semantic name of the needed vertex buffer's value.

Screenshots
https://i.imgur.com/wCtr5ox.png

Additional context
Since i'm not familiar with big C# projects it's hard for me to find the cause in the source code, i will try to have a better look sooner.

@jb-perrier jb-perrier changed the title Missing field " Name " for Custom Vertex Stream in Property Editor [Editor] Missing field " Name " for Custom Vertex Stream in Property Editor Sep 27, 2018
@jb-perrier jb-perrier changed the title [Editor] Missing field " Name " for Custom Vertex Stream in Property Editor [Editor] Missing field " Name " for Custom Vertex Stream in Material Property Editor Sep 27, 2018
@xen2 xen2 added bug Something isn't working area-GameStudio labels Oct 2, 2018
@jb-perrier
Copy link
Author

jb-perrier commented Oct 13, 2018

@xen2
Deleting [InlineProperty] in ComputeVertexStreamBase, fixed the issue. It's kind of a workaround since the engine seems to handle correctly others inlined properties ( the real bug maybe there ).

But once this issue fixed , i run into another crash. When using normal vertex stream, the engine failed to cast Float4 to Float3 , since in shaders normal stream is a Float3. I will open an issue for that.

@xen2
Copy link
Member

xen2 commented Oct 25, 2018

@JimiVacarians ok thanks for the update.
This can work as a temp fix, but InlineProperty will need further investigation

@xen2
Copy link
Member

xen2 commented Oct 25, 2018

@JimiVacarians also please open another issue for the custom vertex stream (we might want an option to choose float3 vs float4).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants