pvanallen/delft-ai-toolkit

Current version of xNode causes bug

pvanallen opened this issue · 5 comments

The system is currently 20 commits back from xNode. In a test, if updated to the current xNode, it breaks ActionsEditor.cs at line 48 with two errors:

Assets/Scripts/delftToolkit/Nodes/Editor/ActionsEditor.cs(48,24): error CS1502: The best overloaded method match for `XNodeEditor.NodeEditorGUILayout.InstancePortList(string, System.Type, UnityEditor.SerializedObject, XNode.NodePort.IO, XNode.Node.ConnectionType, XNode.Node.TypeConstraint, System.Action<UnityEditorInternal.ReorderableList>)' has some invalid arguments

Assets/Scripts/delftToolkit/Nodes/Editor/ActionsEditor.cs(48,135): error CS1503: Argument #6' cannot convert method group' expression to type `XNode.Node.TypeConstraint'

@Siccity can you create a branch to fix this so I can test the whole system with the updated xNode?

Yes I'll do that right away :)

Pushed branch https://github.com/pvanallen/delft-toolkit-v2/tree/xnode-update
It was just a missing extra parameter relating to a new feature. It is now a very simple task to make inputs only accept connections from matching output types. I set the actions list to 'strict', meaning it will only accept action inputs.

Okay, tests out fine. Go ahead and merge with master.

Done