Node Wrangler: Mix related operators are not updated for new Mix node #103912

Closed
opened 2023-01-16 11:06:01 +01:00 by Sun Kim · 6 comments
Contributor

System Information
Operating system: Windows-10-10.0.19045-SP0 64 Bits
Graphics card: Radeon RX 570 Series ATI Technologies Inc. 4.5.0 Core Profile Context 22.11.2.221130

Blender Version
Broken: version: 3.5.0 Alpha, branch: master, commit date: 2023-01-16 00:15, hash: blender/blender@b038662887
Worked: n/a

Addon Information
Name: Node Wrangler (3, 43)
Author: Bartek Skorupa, Greg Zaal, Sebastian Koenig, Christian Brinkmann, Florian Meyer

Short description of error
Mix related operators are not updated for new Mix node. Most notably Lazy Mix still creates the legacy Mix node.

{F14152641, size=full}

{F14152662, size=full}

**System Information** Operating system: Windows-10-10.0.19045-SP0 64 Bits Graphics card: Radeon RX 570 Series ATI Technologies Inc. 4.5.0 Core Profile Context 22.11.2.221130 **Blender Version** Broken: version: 3.5.0 Alpha, branch: master, commit date: 2023-01-16 00:15, hash: `blender/blender@b038662887` Worked: n/a **Addon Information** Name: Node Wrangler (3, 43) Author: Bartek Skorupa, Greg Zaal, Sebastian Koenig, Christian Brinkmann, Florian Meyer **Short description of error** Mix related operators are not updated for new Mix node. Most notably Lazy Mix still creates the legacy Mix node. {[F14152641](https://archive.blender.org/developer/F14152641/NodeWrangerMixFunctions.png), size=full} {[F14152662](https://archive.blender.org/developer/F14152662/NodeWrangerMixFunctions2.png), size=full}
Author
Contributor

Added subscriber: @persun

Added subscriber: @persun

Added subscriber: @mano-wii

Added subscriber: @mano-wii

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

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

I can confirm that the Node Wrangler still adds the "Mix (Legacy)" node instead of the new one.

I can confirm that the Node Wrangler still adds the "Mix (Legacy)" node instead of the new one.
Blender Bot added
Status
Resolved
and removed
Status
Confirmed
labels 2023-02-20 12:15:09 +01:00
Author
Contributor

@wannes.malfait

Hi, thanks for the patch, but it only fixes the merge function. NWBatchChangeNodes and NWChangeMixFactor are still dealing with the old MixRGB node.

@wannes.malfait Hi, thanks for the patch, but it only fixes the merge function. `NWBatchChangeNodes` and `NWChangeMixFactor` are still dealing with the old MixRGB node.
Member

@wannes.malfait

Hi, thanks for the patch, but it only fixes the merge function. NWBatchChangeNodes and NWChangeMixFactor are still dealing with the old MixRGB node.

Yeah, you are right. I missed these. I created another pull request to fix these: #104427

> @wannes.malfait > > Hi, thanks for the patch, but it only fixes the merge function. `NWBatchChangeNodes` and `NWChangeMixFactor` are still dealing with the old MixRGB node. Yeah, you are right. I missed these. I created another pull request to fix these: https://projects.blender.org/blender/blender-addons/pulls/104427
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 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#103912
No description provided.