MO,
Thank you for all your recent node contributions. It is gratifying to have the functionality of NE extended, however I think it is important to resist redundancy where at all possible.
You were perhaps unaware that there was a "Fatlines" node in existence contributed by Wayne. I see that you have added 2 speed modes, which may offer performance / fidelity improvements. Wayne installed his node under Construct2 directory which I think is the appropriate address.
As well, determining the average of a numArray is already provided by Max's "Extract" node. Could "Sum" be added to the "Extract" drop down menu?
As for your "planeCenter" node; this is essentially a replica of Kirsten's "SubD/SelCenterPlane" which is counterintuitively located in SubD directory; Solids is the appropriate directory. However, Wayne made a small modification to Curves/Rectangle node to output surfaces as well as outlines. I have found that in doing so, "SubD/SelCenterPlane" was already redundant. I am not sure if Wayne released Curves2/Rectangle2 to the forum, I believe he did, but anyway it is an elegant & coherent method to generating a plane. This is the perfect example of adding functionality to NE without adding bloat.
Your "RandInRange" has all the functionality of the original "Random" node, yet with the facility to operate within a chosen range and in so it is a worthy replacement.
I made some minor changes to the node's defaults so that it operates similarly to "Random" without connecting inputs. There are many application where only a single floating point number between 0 - 1 is required on running the node, as a consequence I have also made "digital" the default mode. I have attached revised code below, for your consideration.
I hope this post does not read as being critical of your contributions, quite the opposite; I am very happy that you have picked up the mantle, but only that it would be a better time investment to not be reinventing the wheel.
Keep up the good work,
James
https://www.instagram.com/nodeology/

|