Page MenuHome

Geometry Node: Material Count node
Needs ReviewPublic

Authored by Iliya Katueshenock (Moder) on Jun 11 2022, 8:48 PM.
Subscribers
None
Tokens
"Like" token, awarded by IPv6."Like" token, awarded by michaelknubben."Like" token, awarded by eobet."Like" token, awarded by Tolkfan."Love" token, awarded by mindinsomnia."Like" token, awarded by Endertainer007."Love" token, awarded by hitrpr."Yellow Medal" token, awarded by sanek2005.

Details

Summary

Sometimes the user wants to use nodes to work with multiple materials.
This package makes it possible to know about the number of materials without the use of statistics or accumulation.

Currently only mesh is supported, but may be expanded in the future if other components have materials.

Diff Detail

Event Timeline

Iliya Katueshenock (Moder) requested review of this revision.Jun 11 2022, 8:48 PM
Iliya Katueshenock (Moder) created this revision.
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) planned changes to this revision.EditedJun 11 2022, 8:57 PM

Problems need a solution
I look forward to discussing this.

Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)

Other domains seem to have extremely curtailed materials support ...

source/blender/nodes/geometry/nodes/node_geo_find_material.cc
33 ↗(On Diff #52447)

Need to explicitly trim the length of the string to the limit.

Correction:
Now there are no problems when copying materials.
Name string is truncated for search

Resource ownership. Today I realized that this is a fairly common problem.
The materials will only be owned if the user selects them in the field.
If the material was obtained in some other way, then it will not be saved.
I thought I could do ownership. But for this I need to be able to refuse them,
otherwise with each update I will simply produce users.
You need to decide whether geometry nodes should
set ownership of the resources that are computed in
them. Or is it solely the task of the user.

Iliya Katueshenock (Moder) planned changes to this revision.Jun 20 2022, 11:02 AM
Iliya Katueshenock (Moder) retitled this revision from Geometry Node: Add: Material: Info, Count, Find, At Index, Copy, Clear Nodes to Geometry Node: Add: Material: Count, Copy, At Index Nodes.Jun 28 2022, 2:35 PM
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) retitled this revision from Geometry Node: Add: Material: Count, Copy, At Index Nodes to Geometry Node: Add Material Count, Material Copy, Material At Index Nodes.
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
  • Changes
  • Merge master
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) planned changes to this revision.Nov 19 2022, 7:10 PM
  • Merge master
  • Cleanup
  • Input Material Count -> Material Count, now it mesh field input
Iliya Katueshenock (Moder) planned changes to this revision.Nov 27 2022, 11:22 PM
Iliya Katueshenock (Moder) planned changes to this revision.Dec 5 2022, 10:13 PM

I think I'll still split this patch for individual nodes.

Iliya Katueshenock (Moder) retitled this revision from Geometry Node: Add Material Count, Material Copy, Material At Index Nodes to Geometry Node: Material Count node.
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
Iliya Katueshenock (Moder) set the repository for this revision to rB Blender.
  • Merge master
  • Simplifie patch
Iliya Katueshenock (Moder) edited the summary of this revision. (Show Details)
  • Merge master