Create nodes to get/set builtin attributes #91780

Closed
opened 2021-09-28 14:51:52 +02:00 by Jacques Lucke · 12 comments
Member
  • Curve Radius
  • Curve Tilt
  • Curve Handles
  • Shade Smooth
  • Spline Resolution
  • Cyclic Spline
  • Material Index (later)
* Curve Radius * Curve Tilt * Curve Handles * Shade Smooth * Spline Resolution * Cyclic Spline * Material Index (later)
Author
Member

Added subscriber: @JacquesLucke

Added subscriber: @JacquesLucke

Added subscriber: @randum

Added subscriber: @randum

Added subscriber: @guitargeek

Added subscriber: @guitargeek
Johnny Matthews self-assigned this 2021-09-28 21:19:51 +02:00

I'll take this one

I'll take this one
Member

Added subscriber: @HooglyBoogly

Added subscriber: @HooglyBoogly
Member

A note about curve handles:

  • Handle types are already set by D12526
  • Positions need D12005. The node should only allow setting the left or right position at a time, otherwise it has to make the arbitrary choice of what do do when aligned handle types conflict.

And regarding radius:

  • The same node can be used to set point cloud radius and curve radius
  • The node could go in both categories. That's a first, but I think it makes sense here, since they are two separate things.
A note about curve handles: - Handle types are already set by [D12526](https://archive.blender.org/developer/D12526) - Positions need [D12005](https://archive.blender.org/developer/D12005). The node should only allow setting the left or right position at a time, otherwise it has to make the arbitrary choice of what do do when aligned handle types conflict. And regarding radius: - The same node can be used to set point cloud radius and curve radius - The node could go in both categories. That's a first, but I think it makes sense here, since they are two separate things.
Contributor

Added subscriber: @Eary

Added subscriber: @Eary
Contributor

EDIT: I just realized what Hans meant by "can just enter a value directly into the node", so I am just repeating what he said then, lol. Sorry for that

EDIT: The patch is updated as well so I am deleting this comment here.

EDIT: I just realized what Hans meant by "can just enter a value directly into the node", so I am just repeating what he said then, lol. Sorry for that EDIT: The patch is updated as well so I am deleting this comment here.

Added subscriber: @Leul

Added subscriber: @Leul

on Devtalk there was discussion about making these nodes into one node with a drop-down... considering the length of the nodes list i thought it may be a good idea if you haven't considered it

Here is the Post: https://devtalk.blender.org/t/fields-and-anonymous-attributes-proposal/19450/505

on Devtalk there was discussion about making these nodes into one node with a drop-down... considering the length of the nodes list i thought it may be a good idea if you haven't considered it Here is the Post: https://devtalk.blender.org/t/fields-and-anonymous-attributes-proposal/19450/505

This issue was referenced by c1b4abf527

This issue was referenced by c1b4abf527f102ca20114e8179eca511d90784b3
Author
Member

Changed status from 'Needs Triage' to: 'Resolved'

Changed status from 'Needs Triage' to: 'Resolved'
Sign in to join this conversation.
7 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: blender/blender#91780
No description provided.