Maniphest T48094

Node Wrangler Force Update Traceback Error
Closed, Resolved

Assigned To
Greg Zaal (gregzaal)
Authored By
Aaron Grosz (BlenderHead)
Apr 9 2016, 5:36 PM
Tags
  • Add-ons (Community)
Subscribers
Aaron Carlisle (Blendify)
Aaron Grosz (BlenderHead)
Ajlan Altug (jacobo)
Brandon Hix (Acolyte)
Brendon Murphy (meta-androcto)
Denis Belov (dihotom)
Greg Zaal (gregzaal)
3 More Subscribers
Tokens
"Like" token, awarded by gandalf3.

Description

System Information
Windows 10 64bit, MSI GeForce 730

Blender Version
Broken: 2.77a abf6f08
Worked: 2.77

Short description of error
Using Node Wrangler switch node shortcut shift-s, or menu causes a traceback error as shown below. The replaced node still seems to work fine... Doesn't seem to have problems, although the popup is rather annoying.

Event Timeline

Aaron Grosz (BlenderHead) created this task.Apr 9 2016, 5:36 PM
Aaron Grosz (BlenderHead) raised the priority of this task from to 90.
Aaron Grosz (BlenderHead) updated the task description.
Aaron Grosz (BlenderHead) added a project: Add-ons (Community).
Aaron Grosz (BlenderHead) edited a custom field.
Aaron Grosz (BlenderHead) added a subscriber: Aaron Grosz (BlenderHead).
Aaron Carlisle (Blendify) added a subscriber: Aaron Carlisle (Blendify).EditedApr 9 2016, 8:34 PM

Seems to be caused by rBA1c9d5a8507acfe76b7d4770f8436af0456ee9f55

Aaron Carlisle (Blendify) assigned this task to Greg Zaal (gregzaal).Apr 9 2016, 8:35 PM
Aaron Carlisle (Blendify) lowered the priority of this task from 90 to 50.
jim ww smith (jim_ww) added a subscriber: jim ww smith (jim_ww).Apr 10 2016, 12:31 AM
Aaron Grosz (BlenderHead) added a comment.Apr 10 2016, 4:12 AM

Figured it out.
In line 1897, I changed 'force_update(context)' to 'hack_force_update(context, nodes)' as that was how it was used elsewhere in the file.

Greg Zaal (gregzaal) added a subscriber: Brendon Murphy (meta-androcto).Apr 10 2016, 11:41 AM

hack_force_update is old code, replaced recently by the simpler force_update method. Maybe rBA27da8e1f was lost in the 2.77a release?? *This* is the current version of the add-on straight from git, and it does indeed have the new function.

@Brendon Murphy (meta-androcto) maybe you know what's going on?

Greg Zaal (gregzaal) changed the task status from Unknown Status to Resolved.Apr 10 2016, 12:39 PM

Spoke with Brendon and Cambo on IRC, looks like that commit just got missed for 2.77a. If there's a 2.77b, it'll be fixed in there. Otherwise just download the latest version and replace your current one (/2.77/scripts/addons/node_wrangler.py) with that.

jim ww smith (jim_ww) removed a subscriber: jim ww smith (jim_ww).Apr 11 2016, 2:03 AM
Aaron Grosz (BlenderHead) added a comment.Apr 11 2016, 8:05 PM

Thanks Greg.
It definitely works now. Great tip and addon.

Aaron Grosz (BlenderHead) added a comment.Apr 11 2016, 8:05 PM

Thanks Greg.
It definitely works now. Great tip and addon.

Aaron Grosz (BlenderHead) added a comment.Apr 11 2016, 8:05 PM

Thanks Greg.
It definitely works now. Great tip and addon.

Aaron Carlisle (Blendify) added a subscriber: Ajlan Altug (jacobo).
Greg Zaal (gregzaal) added subscribers: Greg Zaal (gregzaal), Kitt Zwovic (gandalf3), Denis Belov (dihotom).
Kitt Zwovic (gandalf3) awarded a token.Apr 16 2016, 6:27 PM
Aaron Carlisle (Blendify) added a subscriber: Leroy (Leroy).
Martin Hooper (martinjh99) added a subscriber: Martin Hooper (martinjh99).May 21 2016, 11:23 PM

Can someone delete T48488 then as it is a duplicate of this one. Thanks

Vuk Gardašević (lijenstina) mentioned this in T48724: Error with nodewrangler add-on....Jun 24 2016, 12:19 AM
Bastien Montagne (mont29) merged a task: T48724: Error with nodewrangler add-on....Jun 24 2016, 2:51 PM
Bastien Montagne (mont29) added subscribers: Vuk Gardašević (lijenstina), Brandon Hix (Acolyte).