blender/blender-manual
23
23
Fork 104
Code Issues 75 Pull Requests 7 Projects 1 Releases Wiki Activity

adds docs for new geonodes corners_of_edge #104464

Merged
Hans Goudey merged 12 commits from F_Scociety/blender-manual:corners_of_edge into main 2023-06-06 16:06:44 +02:00
Conversation 0 Commits 12 Files Changed 4 +13 -4
2 changed files with 13 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e9259024cf - Show all commits

BIN
manual/images/modeling_geometry-nodes_corners-of-edge_explanation.png Normal file
View File

Binary file not shown.
Side by Side

After

Width:  |  Height:  |  Size: 405 KiB

17
manual/modeling/geometry_nodes/mesh/topology/corners_of_edge.rst
View File

@ -12,12 +12,21 @@ Corners of Edge Node
The *Corners of Edge* node retrieves face corners connected to each edge in a mesh. The *Corners of Edge* node retrieves face corners connected to each edge in a mesh.
The node first gathers a list of face corners which are connected to the edges. The node first gathers a list of face corners which are connected to the edges.
F_Scociety marked this conversation as resolved Outdated
Hans Goudey commented 2023-06-02 00:53:16 +02:00
Outdated
Review
Copy Link

This is a good opportunity to be more specific about which faces are included. Going back to the devtalk thread, the behavior we chose deserves a bit of explanation IMO. Likewise with the fact that only the previous corner is used, not the next. That should be mentioned at least. Though if you're up for making some sort of diagram, that would be great too!

This is a good opportunity to be more specific about which faces are included. Going back to the devtalk thread, the behavior we chose deserves a bit of explanation IMO. Likewise with the fact that only the previous corner is used, not the next. That should be mentioned at least. Though if you're up for making some sort of diagram, that would be great too!
Soeren Schmidt-Clausen commented 2023-06-02 21:37:13 +02:00
Outdated
Review
Copy Link

Yeah, a bit more explanation would definetly help in understanding the node better.

What do you think about this description?
The Corners of Edge node retrieves face corners connected to each edge in a mesh.
The node first gathers a list of the first corner, in winding order, of all faces connected to the edge.
It's important to note that the node only returns the first corner connected to the edge.
By offsetting the index by 1, you can then get the other connected corner to the edge per face.
That list is then sorted based on the values of the Sort Weight input.
The Total output is the number of connected faces/corners, and the Corner Index
output is one of those corners, chosen with the Sort Index input.

According to the diagram: I iam not entirely sure whats the best way to intergrate explanation pictures in the docs. But I created some tests, inspirered by the ones, I did for the devtalk proposial. I prefer the grease pencil one (1) more, because it looks familiar to blenders UI:
(1) : https://ibb.co/VBg7pMk
(2) : https://ibb.co/wK14vKX

Yeah, a bit more explanation would definetly help in understanding the node better. What do you think about this description? The *Corners of Edge* node retrieves face corners connected to each edge in a mesh. The node first gathers a list of the first corner, in winding order, of all faces connected to the edge. It's important to note that the node only returns the first corner connected to the edge. By offsetting the index by 1, you can then get the other connected corner to the edge per face. That list is then sorted based on the values of the *Sort Weight* input. The *Total* output is the number of connected faces/corners, and the *Corner Index* output is one of those corners, chosen with the *Sort Index* input. According to the diagram: I iam not entirely sure whats the best way to intergrate explanation pictures in the docs. But I created some tests, inspirered by the ones, I did for the devtalk proposial. I prefer the grease pencil one (1) more, because it looks familiar to blenders UI: (1) : https://ibb.co/VBg7pMk (2) : https://ibb.co/wK14vKX
Hans Goudey commented 2023-06-02 23:06:41 +02:00
Outdated
Review
Copy Link

Yeah, that's definitely helpful! I like the grease pencil styled one too. I wonder if using some mesh for some things like the circles would help avoid the hand drawn look though.

gathers a list of the first corner, in winding order, of all faces connected

"in winding order" isn't quite right here, since that refers to the sorting of corners around a face.

By offsetting the index by 1

Might be nice to mention (link) the "Offset Corner In Face" node here, since just adding one to the corner index might end up in a different face.

Yeah, that's definitely helpful! I like the grease pencil styled one too. I wonder if using some mesh for some things like the circles would help avoid the hand drawn look though. >gathers a list of the first corner, in winding order, of all faces connected "in winding order" isn't quite right here, since that refers to the sorting of corners around a face. >By offsetting the index by 1 Might be nice to mention (link) the "Offset Corner In Face" node here, since just adding one to the corner index might end up in a different face.
Soeren Schmidt-Clausen commented 2023-06-05 11:27:51 +02:00
Outdated
Review
Copy Link

Yeah, that's definitely helpful! I like the grease pencil styled one too. I wonder if using some mesh for some things like the circles would help avoid the hand drawn look though.

New Version: https://ibb.co/5cVChYs

"in winding order" isn't quite right here, since that refers to the sorting of corners around a face.

Maybe a better description, but i find the naming very difficult to be precise:

The node first gathers a list of face corners which are connected to the edges.
It's important to note, that the list only has one corner per face.
Offsetting the index inside the face by 1 using the :doc:Offset Corners in Face </modeling/geometry_nodes/mesh/topology/offset_corner_in_face> returns the other corner, of the connected face to the edge.

Might be nice to mention (link) the "Offset Corner In Face" node here, since just adding one to the corner index might end up in a different face.

Yep, I'll be doing that with the next commit

> Yeah, that's definitely helpful! I like the grease pencil styled one too. I wonder if using some mesh for some things like the circles would help avoid the hand drawn look though. New Version: https://ibb.co/5cVChYs > "in winding order" isn't quite right here, since that refers to the sorting of corners around a face. Maybe a better description, but i find the naming very difficult to be precise: The node first gathers a list of face corners which are connected to the edges. It's important to note, that the list only has one corner per face. Offsetting the index inside the face by 1 using the :doc:`Offset Corners in Face </modeling/geometry_nodes/mesh/topology/offset_corner_in_face>` returns the other corner, of the connected face to the edge. > Might be nice to mention (link) the "Offset Corner In Face" node here, since just adding one to the corner index might end up in a different face. Yep, I'll be doing that with the next commit
Hans Goudey commented 2023-06-05 17:09:20 +02:00
Outdated
Review
Copy Link

That sounds great! And the image looks great too.

Small thing: there's no need for the comma after "It's important to note".

That sounds great! And the image looks great too. Small thing: there's no need for the comma after "It's important to note".
Soeren Schmidt-Clausen commented 2023-06-05 17:45:17 +02:00
Outdated
Review
Copy Link

Great!
Corrected it.

Do you have a tip on how I can best intergrade the image to comply with the docs? Do you know an example in the docs where a extra explanation image was added?

Great! Corrected it. Do you have a tip on how I can best intergrade the image to comply with the docs? Do you know an example in the docs where a extra explanation image was added?
Hans Goudey commented 2023-06-05 20:34:09 +02:00
Outdated
Review
Copy Link

The extrude node is the most recent one I worked on. Other geometry nodes have an "Examples" section at the bottom. That's not really an example though, so it's probably fine if it goes in the first section before the "Inputs".

The extrude node is the most recent one I worked on. Other geometry nodes have an "Examples" section at the bottom. That's not really an example though, so it's probably fine if it goes in the first section before the "Inputs".
Soeren Schmidt-Clausen commented 2023-06-06 11:18:16 +02:00
Outdated
Review
Copy Link

Thanks, the extrude node helped a lot!

Thanks, the extrude node helped a lot!
It's important to note that the list only has one corner per face. It's important to note that the list only has one corner per face.
Offsetting the index inside the face by 1 using the :doc:`Offset Corners in Face </modeling/geometry_nodes/mesh/topology/offset_corner_in_face>` Using the :doc:Offset Corners in Face </modeling/geometry_nodes/mesh/topology/offset_corner_in_face>
F_Scociety marked this conversation as resolved Outdated
Hans Goudey commented 2023-06-06 14:06:34 +02:00
Outdated
Review
Copy Link

:doc:Offset Corners in Face </modeling/geometry_nodes/mesh/topology/offset_corner_in_face> -> :doc:/modeling/geometry_nodes/mesh/topology/offset_corner_in_face

The name, along with "node" should be filled in automatically.

`:doc:Offset Corners in Face </modeling/geometry_nodes/mesh/topology/offset_corner_in_face>` -> `:doc:/modeling/geometry_nodes/mesh/topology/offset_corner_in_face` The name, along with "node" should be filled in automatically.
returns the other corner, of the connected face to the edge. to offset the index inside the face by 1, returns the other corner from the face that is connected to the edge
F_Scociety marked this conversation as resolved Outdated
Hans Goudey commented 2023-06-06 14:05:48 +02:00
Outdated
Review
Copy Link

, returns -> gives (more friendly word for non-programmers. The comma is unecessary too.

`, returns` -> ` gives` (more friendly word for non-programmers. The comma is unecessary too.
Hans Goudey commented 2023-06-06 14:38:34 +02:00
Outdated
Review
Copy Link

Missing period at the end of this sentence.

Missing period at the end of this sentence.
That list is then sorted based on the values of the *Sort Weight* input. That list is then sorted based on the values of the *Sort Weight* input.
The *Total* output is the number of connected faces/corners, and the *Corner Index* The *Total* output is the number of connected faces/corners, and the *Corner Index*
output is one of those corners, chosen with the *Sort Index* input. output is one of those corners, chosen using the *Sort Index* input.
.. figure:: /images/modeling_geometry-nodes_corners-of-edge_explanation.png
:align: center
:width: 400px
A graphic for which corners are returned for a given edge
* red: selected edge;
F_Scociety marked this conversation as resolved Outdated
Hans Goudey commented 2023-06-06 14:07:22 +02:00
Outdated
Review
Copy Link

Capitalize these colors. No need for the semicolon here also IMO

Capitalize these colors. No need for the semicolon here also IMO
Hans Goudey commented 2023-06-06 14:38:50 +02:00
Outdated
Review
Copy Link

Sorry, I meant like this :P

Red, Blue, Purple
Sorry, I meant like this :P ``` Red, Blue, Purple ```
* blue: corners that get individually returned, depending on the sorting
* purple: corners that can be retrieved by offseting the blue corner indices using :doc:`Offset Corners in Face </modeling/geometry_nodes/mesh/topology/offset_corner_in_face>`
Inputs Inputs
====== ======
@ -49,7 +58,7 @@ Outputs
======= =======
Corner Index Corner Index
An corner of the input edge in its face's winding order, chosen by the *Sort Index* input. A corner of the input edge in its face's winding order, chosen by the *Sort Index* input.
Total Total
The number of faces or face corners connected to the edge. The number of faces or face corners connected to the edge.