WebGL Bug with VariableSeperator

[patientzerogames]

Hi

just wanted to let you know that there seems to be a bug when using Nodecanvas in a WebGL build.
It is caused by the VariableSeperator and throws the following error:

b>(Deserialization Error)</b>: Constructor of NodeCanvas.Framework.Variable1[NodeCanvas.Framework.VariableSeperator] threw an exception when creating an instance
at ParadoxNotion.Serialization.FullSerializer.fsMetaType.CreateInstance () [0x00000] in <filename unknown>:0
at ParadoxNotion.Serialization.FullSerializer.Internal.fsReflectedConverter.CreateInstance (ParadoxNotion.Serialization.FullSerializer.fsData data, System.Type storageType) [0x00000] in <filename unknown>:0`

Adding a #if UNITY_WEBGL in the Variable.cs class seems to fix the error

1 2 3 4 5 6 7 8

/////This is a very special dummy struct for variable header separators     public struct VariableSeperator     { #if UNITY_EDITOR || UNITY_WEBGL         [NonSerialized]         public bool isEditingName; #endif     }

[Gavalakis]

Hey,
I’m very sorry for the late reply, but I was ill for the last couple of days.
Thanks a lot for the bug report and fix!
I will also investigate further to find the real cause of this 🙂

Join us on Discord: https://discord.gg/97q2Rjh

[patientzerogames]

Hey,
No problem!

I came across another small bug.
If you use the Control Graph Owner Task triggered by a ConcurentState or AnyState to stop a FSM it will cause a Nullreference exception in the FSM.OnGraphUpdate() method

1 2 3 4 5 6 7

//Update AnyStates and ConcurentStates for (var i = 0; i < updatableNodes.Count; i++){ updatableNodes<em class="d4pbbc-italic"></em>.Update(); }   //Update current state (this needs a null check, since the for loop above can cause the currentState to become null) currentState.Update();

[Gavalakis]

Ah. Thanks for that too 🙂
The problem is more relevant to GraphOwnerControl task. Here is a fixed version. A bit tricky 🙂

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61

using NodeCanvas.Framework; using ParadoxNotion.Design; using System.Collections;   namespace NodeCanvas.Tasks.Actions{   [Name("Control Graph Owner")] [Category("✫ Utility")] [Description("Start, Resume, Pause, Stop a GraphOwner's behaviour")] public class GraphOwnerControl : ActionTask<GraphOwner> {   public enum Control { StartBehaviour, StopBehaviour, PauseBehaviour }   public Control control = Control.StartBehaviour; public bool waitActionFinish = true;   protected override string info{ get {return agentInfo + "." + control.ToString();} }   protected override void OnExecute(){ StartCoroutine(Do()); }   IEnumerator Do(){ yield return null;   if (control == Control.StartBehaviour){ if (waitActionFinish){ agent.StartBehaviour( (s)=> { EndAction(s); } ); } else { EndAction(); agent.StartBehaviour(); } }   if (control == Control.StopBehaviour){ EndAction(); agent.StopBehaviour(); }   if (control == Control.PauseBehaviour){ EndAction(); agent.PauseBehaviour(); } }   protected override void OnStop(){ if (waitActionFinish && control == Control.StartBehaviour){ agent.StopBehaviour(); } } } }

Thanks 🙂

Join us on Discord: https://discord.gg/97q2Rjh

[patientzerogames]

Great!
Thx for the quick fix!

[Author]

Posts