BlueGraph icon indicating copy to clipboard operation
BlueGraph copied to clipboard

Open graph loses connections after reload

Open AncientPixel opened this issue 1 year ago • 1 comments

I love BlueGraph for its simplicity and ease of use but still being customisable. An issue I have with it in Unity 2023 is that if a graph is open and I make a code change the connections of the graph get dropped with an error to the log. This is in the CanvasView 'rebuild' of node, port and connection views.

Error: Could not connect Journal Page:ExecIn: Connected node no longer exists. UnityEngine.Debug:LogError (object) BlueGraph.Editor.CanvasView:AddNodeViews (System.Collections.Generic.IEnumerable1<BlueGraph.Node>,bool,bool) (at Assets/Scripts/BlueGraph/Editor/CanvasView.cs:631)

It seems during recompiling the reference to the port in the Connection class is lost but never reestablished.

To fix this, in the CanvasView.cs file in the AddNodeViews function add port.UpdateConnections(); around line ~610 just below foreach (var port in node.Key.Ports.Values).

AncientPixel avatar Oct 24 '23 12:10 AncientPixel

I love BlueGraph for its simplicity and ease of use but still being customisable. An issue I have with it in Unity 2023 is that if a graph is open and I make a code change the connections of the graph get dropped with an error to the log. This is in the CanvasView 'rebuild' of node, port and connection views.

Error: Could not connect Journal Page:ExecIn: Connected node no longer exists. UnityEngine.Debug:LogError (object) BlueGraph.Editor.CanvasView:AddNodeViews (System.Collections.Generic.IEnumerable1<BlueGraph.Node>,bool,bool) (at Assets/Scripts/BlueGraph/Editor/CanvasView.cs:631)

It seems during recompiling the reference to the port in the Connection class is lost but never reestablished.

To fix this, in the CanvasView.cs file in the AddNodeViews function add port.UpdateConnections(); around line ~610 just below foreach (var port in node.Key.Ports.Values).

This doesn't seem to fix the problem for me.

TehMighty avatar Feb 17 '24 06:02 TehMighty