This patch adds a node, that removes a custom attribute.
Details
Details
Diff Detail
Diff Detail
- Repository
- rB Blender
Event Timeline
Comment Actions
Nice and simple!
| source/blender/nodes/geometry/nodes/node_geo_attribute_remove.cc | ||
|---|---|---|
| 17–22 ↗ | (On Diff #35077) | Unused includes |
| 42–43 ↗ | (On Diff #35077) | This is a common, expected situation, no need to add an error in this case. |
| 47–48 ↗ | (On Diff #35077) | Personally I would skip assigning the temp variable here and just put the operation in the if statement. |
| 48–49 ↗ | (On Diff #35077) | I'd like to be explicit and add the name of the attribute here, like the other related error messages |
Comment Actions
Is it possible to have the Attribute input socket as a multi input socket? Some of the tools I've been making have up to 23 attributes that would need clearing out so accepting multiple string nodes would really streamline this
Comment Actions
That is something we plan on adding. It would require a new multiple-input socket, similar to the switch node design
Comment Actions
That is something we plan on adding. It would require a new multiple-input socket, similar to the switch node design
@Eitan Traurig (EitanSomething) The new multi-input socket with sorting is already in master. You can use it for the switch node.