Jump to content

MarbleProceduralTexture marbleClolor not defined


Recommended Posts


I miss a declaration and implementation of the property "marbleClolor" in 3..0-alpha, but the shader expects an uniform variable.

In old soures this property  was defined, is it obsolete now? But nevertheless, there is an iconsistency between shader and js/ts


Link to comment
Share on other sites

Hmm, problems here:

1) where to find previous releases (not babylon, but procedural_textures)?

2) as I can see on my old private sources, "marbleColor" was never used, it's only declared on the fragment shader. The color for the marble is coming from the shader itself (a synthetic generated blackwhite-color). One way to input a user color would be to mix the generated color with the user-color, which results in an output-contrast of  50% if the variable is not specified, which would not be fine.

So my suggestion conclusion is NOT to modify the shader, because of side effects. The user can still mix color via diffuse/emissiveColor etc.  I also would suggest to leave the variable in the fragment shader for the case someone has an innovative idea to upgrade this shader. "Never touch a running System" :)

If you agree, we can then set this bug-report to solved, sorry for my hasty thread :(

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Recently Browsing   0 members

    • No registered users viewing this page.
  • Create New...