I put the node into a new sub menu called "Volume".
Ref T84606.
Details
Details
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- temp-geometry-nodes-points-to-volume (branched from master)
- Build Status
Buildable 12313 Build 12313: arc lint + arc unit
Event Timeline
Comment Actions
I found that the node has no output if the radius is smaller than the calculated voxel size (or the direct input voxel size in "size" mode). I think that's expected, but I suppose that's a case for a warning message.
Actually, even this case where the radius is larger than the voxel size has no output (on a 96m tall cube with subdiv level 6 applied):
Not for this patch, but I wonder if in combination with the volume to mesh node, this might eventually need a some sort of "max grid size" value so we don't melt people's computers. I'm surprised how well this performs actually.
My inline comments are mostly suggestions for tweaks to UI descriptions.
| source/blender/makesrna/intern/rna_nodetree.c | ||
|---|---|---|
| 8900 | Desired number of voxels along one axis -> Specify the numer of voxels along each axis | |
| 8905 | Desired voxel side length -> Specify the voxel side length | |
| 8914 | Mode for how the desired voxel size is specified -> How the voxel size is specified | |
| source/blender/nodes/geometry/nodes/node_geo_points_to_volume.cc | ||
| 135 | I also don't see a need for a default value here. Not sure in what case that would fail anyway. | |
| 174 | 0.0f | |
| 182–188 | What's the idea behind creating a second grid and then merging it with the first? Maybe that deserves a comment. | |
Comment Actions
Works great! I don't see any problems with it.
I mentioned this before but just want to reiterate it here:
I can see two things that would be nice to add down the line
- A falloff parameter
- Different modes to handle intersection of points. Might be nice to allow an additive mode as well for example
These things don't matter for this first iteration on the node, just wanted to have them written down here.
| source/blender/nodes/geometry/nodes/node_geo_points_to_volume.cc | ||
|---|---|---|
| 135 | The attribute getter methods without "try" require some kind of default in case there is an error. I could also assert, but I think I'd leave it as is. | |
| 182–188 | I'll add a comment mentioning that the merge is cheap. In this specific case I could use the existing grid, but that could change if the algorithm needs to be adapted. Furthermore, that makes the generate_volume_from_points more self contained, because it does not have to worry about data that might exist in the incoming grid already. | |
Comment Actions
Looks good to me!
The cropped text is not a unique problem, many of the new nodes have it, but I fount that node_type_size(&ntype, 170, 120, 700); gets the words to fit if you'd like to use that.
| source/blender/nodes/geometry/nodes/node_geo_points_to_volume.cc | ||
|---|---|---|
| 154 | Hmm, maybe a more detailed comment about why subtracting .5 makes the alignment better? | |
| 213–227 | We should probably settle on some standard for whether to put these functions at the top or bottom of the file. I think most of the more recent additions have been putting them at the top?] | |