Node Wrangler: Uncorrect lazy connection for switch nodes and multi input #103205

Closed
opened 2022-12-14 09:04:21 +01:00 by bigmark · 13 comments

System Information
Operating system: Windows-10-10.0.22621-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 2060 SUPER/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 527.56

Blender Version
Broken: version: 3.5.0 Alpha, branch: master, commit date: 2022-12-13 01:46, hash: blender/blender@e6e57cebec
Worked: (newest version of Blender that worked as expected)

Short description of error
[Please fill out a short description of the error here]

Exact steps for others to reproduce the error
go to the geometry nodes tab and create a new geometry nodes setup, then create 2 switches and plug the output of each into a "join geometry" node. then plug the join geometry node to the group output "geometry" node, and boom. it crashes.

**System Information** Operating system: Windows-10-10.0.22621-SP0 64 Bits Graphics card: NVIDIA GeForce RTX 2060 SUPER/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 527.56 **Blender Version** Broken: version: 3.5.0 Alpha, branch: master, commit date: 2022-12-13 01:46, hash: `blender/blender@e6e57cebec` Worked: (newest version of Blender that worked as expected) **Short description of error** [Please fill out a short description of the error here] **Exact steps for others to reproduce the error** go to the geometry nodes tab and create a new geometry nodes setup, then create 2 switches and plug the output of each into a "join geometry" node. then plug the join geometry node to the group output "geometry" node, and boom. it crashes.
Author

Added subscriber: @bigmark

Added subscriber: @bigmark

Added subscriber: @mod_moder

Added subscriber: @mod_moder

I'll check

I'll check

Changed status from 'Needs Triage' to: 'Needs User Info'

Changed status from 'Needs Triage' to: 'Needs User Info'

Hello. I can't confirm the crash if you follow your steps.
I found a crash related to changing the switch type. But I don't see it in your description. Is this what you want to report, or can you record a video example of your description?

Hello. I can't confirm the crash if you follow your steps. I found a crash related to changing the switch type. But I don't see it in your description. Is this what you want to report, or can you record a video example of your description?

I found this while reviewing your report. If this is what you found, it will be possible to close it as a duplicate.
blender/blender#103208 (Geometry Node: Multi input socket cause crash if change type of connected socket)

I found this while reviewing your report. If this is what you found, it will be possible to close it as a duplicate. blender/blender#103208 (Geometry Node: Multi input socket cause crash if change type of connected socket)
Author

In #103205#1460069, @mod_moder wrote:
Hello. I can't confirm the crash if you follow your steps.
I found a crash related to changing the switch type. But I don't see it in your description. Is this what you want to report, or can you record a video example of your description?

2022-12-14 03-12-00.mp4 yes, here you go

> In #103205#1460069, @mod_moder wrote: > Hello. I can't confirm the crash if you follow your steps. > I found a crash related to changing the switch type. But I don't see it in your description. Is this what you want to report, or can you record a video example of your description? [2022-12-14 03-12-00.mp4](https://archive.blender.org/developer/F14058359/2022-12-14_03-12-00.mp4) yes, here you go

Maybe try to repeat this without the included addon for quick link connection?
I see that the problem is that both links have 0 index

Maybe try to repeat this without the included addon for quick link connection? I see that the problem is that both links have 0 index

Changed status from 'Needs User Info' to: 'Needs Triage'

Changed status from 'Needs User Info' to: 'Needs Triage'

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

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

I see what the problem is.
This was partly fixed by my patch yesterday, but the addon still doesn't connect nodes correctly on this operation.

I see what the problem is. This was partly fixed by my patch yesterday, but the addon still doesn't connect nodes correctly on this operation.
Iliya Katushenock changed title from geometry nodes bug! to Node Wrangler: Uncorrect lazy connection for switch nodes and multi input 2022-12-15 06:44:45 +01:00

Closed as duplicate of blender/blender#103395

Closed as duplicate of blender/blender#103395

I see that the node wrangler addon issue is the same as in other reports.
So I will salt them as 1.
This is a known issue (wrong connection, ignoring the need to index visible sockets only...)

I see that the node wrangler addon issue is the same as in other reports. So I will salt them as 1. This is a known issue (wrong connection, ignoring the need to index visible sockets only...)
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#103205
No description provided.