NodeMaterial: getHash and customProgramCacheKey

[sunag]

Node.getHash() or NodeMaterial.getHash() generate a JSON.

This approach building the string is more fast that array.join and JSON.stringify.

[Mugen87]

This is great! 👍

@mrdoob When this is merged, it should be okay to merge #19673 (which ensures onBeforeCompile() is called at the correct time and improves the performance of getParameters()).

[sunag]

Node.hashProperties is necessary in case of variation of the shader from the node config.

[mrdoob]

@sunag ready to be merged?

[sunag]

@sunag ready to be merged?

Yes!

[WestLangley]

@mugen said

I can clean things up after merging. Don't be worry about this

Please be consistent. If users are not expected to follow the three.js code style, then please remove the instructions to do so found in https://github.com/mrdoob/three.js/wiki/How-to-contribute-to-three.js.

Format whitespace consistently with the rest of code base. See the [an] article Mr.doob's-Code-Style™, and the online code beautifier. It is highly recommended to use ESLint plugin and appropriate settings in your code editor.

/ping @mrdoob

[sunag]

@WestLangley I see no problem in fix this... so far I thought that you is referring to the json output.

[Mugen87]

Sorry, just wanted to get this merged quickly 🤓.

[mrdoob]

Thanks!