@CharlieJolly
blender::get_default_hash_2<T, int>
(frombli
)
From previous review from Jacque it was decided to expose noise::hash
functions and not default_hash
.
Also this seems a bit strange in comparison with the other types. I'd propose the following: Add a Seed integer field input, same as the random value node has, for all types. That way…
To me it's not immediately obvious, why there are 3 inputs for the Integer type. I can see needing 2, but that should be enough to hash any amount of integers together by folding.
The hash…
hash-to-float
was removed, some changes were discussed in chat but the node didn't have any tlc for a while
The code generally looks good. I think the main thing that's missing in the description is the motivation and examples for how someone should use this. Like, why should this be used instead of…
Ok, I get what you are saying but we're not consistent here. E.g. Map Range doesn't list all available sockets. Listing all sockets also adds noise that the user needs to go through. In most…
I'm still not understanding what this is doing different other than using an array and excluding Rotation socks.
It might be easier for you to provide code if you need me to change something here.
Or are you talking about weight as is done for math node?
Sorry, I still don't understand. Drag link only limits by type not node. Hash output is Integer. User might want to plug that into a math node.
I'm guessing node vs modes is no longer a proposal. This creates a UI divergence with existing nodes.
Is the Swap Node Operator #111438 going to be a target for 4.1?
![image](/attachments/e6…
For reference, I previously created a similar node Add Random Spherical Distribution node which had controls to limit the radius, angles. (See https://archive.blender.org/developer/D12746). This…