Scale vertexIntensity values by max value encountered

[etpinard]

to bring gl-streamtube3d to par with gl-cone3d and make the plotly.js wrapper a lot less painful to write.

I hope @kig won't mind.

[kig]

I was going to suggest dealing with division by zero here, but on second thought, it's not necessary. With zero field, you'd end up with a division by zero. But with a zero field you wouldn't have a stream anyway, so you shouldn't encounter this line with zero maxNorm.

[kig]

This is a per-stream maxNorm. You probably want to compute a maxNorm of all the streams instead and pass it to streamToTube.

[kig]

Thanks, this is good to have.

If need be, we could also add something like an intensityRange: [min, max] config parameter that would be used to map velocity values to 0..1 range.

[etpinard]

Thanks for fixing this up @kig 🎉

If need be, we could also add something like an intensityRange: [min, max] config parameter that would be used to map velocity values to 0..1 range.

We could, but we don't need to add this to ./streamtube.js. Setting vertexIntensityBounds on the meshData work fine too e.g:

var meshData = tube2mesh({/* */})
meshData.vertexIntensityBounds = [/**/, /**/]
createTubeMesh(meshData)