Regression: 3.4 mix colour node. links.new() not working properly via python #104415

New Issue

Malcolm Reed · 2023-02-07T18:08:22+01:00

Malcolm Reed commented

2023-02-07 18:08:22 +01:00

System Information
Operating system: windows 10
Graphics card: Nvidia Geforce rtx

Blender Version
Broken: 3.4.1
Worked: 3.3.1
Short description of error
Making a link to index 1 or 2 via python on the mix colour node doesn't properly connect.
If queried python reports a connection, but there is no noodle or visible output.

Exact steps for others to reproduce the error
Please see the attached files.
In 3.4.1 load: MixColor_node_links_3_4_1.blend
Select the sphere.
Run the script.
Look in the system console for reporting.

 - Input 1 is free
 - Yes a valid link
 - Input 1 is busy

This is saying that the input [1] on the node is free before the link is attempted.
After, tests that the link that is made is valid.
Says that the input is in use.

However you'll see that their is no connection between the newly created Texture node and the input[1] on the mix colour node.

If you then run it again immediately you'll see this:

 - Input 1 is busy
 - Yes a valid link
 - Input 1 is busy

This time it reports that the link is busy, although visually there is no connection.

Delete the new Texture nodes and we're back to the first result.

If you were to change line 43

baseCol_socket = mixAO.inputs[1]

to

baseCol_socket = mixAO.inputs[0]

Then the connection can be made to the factor input.

It seems their is some logical connection made, however the noodle isn't drawn.

To test in blender 3.3.1, load the file: MixColor_node_links_3_3_1.blend

**System Information** Operating system: windows 10 Graphics card: Nvidia Geforce rtx **Blender Version** Broken: 3.4.1 Worked: 3.3.1 **Short description of error** Making a link to index 1 or 2 via python on the mix colour node doesn't properly connect. If queried python reports a connection, but there is no noodle or visible output. **Exact steps for others to reproduce the error** Please see the attached files. In 3.4.1 load: MixColor_node_links_3_4_1.blend Select the sphere. Run the script. Look in the system console for reporting. ``` - Input 1 is free - Yes a valid link - Input 1 is busy ``` This is saying that the input [1] on the node is free before the link is attempted. After, tests that the link that is made is valid. Says that the input is in use. However you'll see that their is no connection between the newly created Texture node and the input[1] on the mix colour node. If you then run it again immediately you'll see this: ``` - Input 1 is busy - Yes a valid link - Input 1 is busy ``` This time it reports that the link is busy, although visually there is no connection. Delete the new Texture nodes and we're back to the first result. If you were to change line 43 ``` baseCol_socket = mixAO.inputs[1] ``` to ``` baseCol_socket = mixAO.inputs[0] ``` Then the connection can be made to the factor input. It seems their is some logical connection made, however the noodle isn't drawn. To test in blender 3.3.1, load the file: MixColor_node_links_3_3_1.blend

MixColor_node_links_3_4_1.blend

1.6 MiB

MixColor_node_links_3_3_1.blend

1.5 MiB

Malcolm Reed added the

labels 2023-02-07 18:08:23 +01:00

Scurest commented

2023-02-07 18:37:39 +01:00

(There have been several issues about this but I can't find them now since search doesn't seem to work.)

The Mix node now has 8 inputs (and 3 outputs):

>>> for i, input in enumerate(mix_node.inputs):
...     print(i, input.name, input.identifier)
...     
0 Factor Factor_Float
1 Factor Factor_Vector
2 A A_Float
3 B B_Float
4 A A_Vector
5 B B_Vector
6 A A_Color
7 B B_Color

This is to support being able to mix floats/vecs/colors with the same node. The inputs that are not used by the current mode are just hidden. In Color mode, the only visible inputs are 0, 6, and 7.

You're connecting inputs[1], which is the Factor_Vector input. And indeed, if you change the Mix node to Vector+Nonuniform mode, you will see the link you just created going to the texture node.

What you probably want, though, is inputs[6].

(There have been several issues about this but I can't find them now since search doesn't seem to work.) The Mix node now has 8 inputs (and 3 outputs): ``` >>> for i, input in enumerate(mix_node.inputs): ... print(i, input.name, input.identifier) ... 0 Factor Factor_Float 1 Factor Factor_Vector 2 A A_Float 3 B B_Float 4 A A_Vector 5 B B_Vector 6 A A_Color 7 B B_Color ``` This is to support being able to mix floats/vecs/colors with the same node. The inputs that are not used by the current mode are just hidden. In Color mode, the only visible inputs are 0, 6, and 7. You're connecting inputs[1], which is the Factor_Vector input. And indeed, if you change the Mix node to Vector+Nonuniform mode, you _will_ see the link you just created going to the texture node. What you probably want, though, is inputs[6].

Iliya Katushenock commented

2023-02-07 18:44:04 +01:00

I close this as duplicate of: #103395

I close this as duplicate of: https://projects.blender.org/blender/blender/issues/103395

Iliya Katushenock closed this issue

2023-02-07 18:52:24 +01:00

Blender Bot added

and removed