Graph runtime installation with settings

[slava]

Hello there!

I have an issue with runtime installation of the Graphs. I’ve noticed that Initilize() function is called only once in the GraphOwner. So it means that if I set “preinitialize subgraphs” to true and I will have a graph assigned in the editor already or if I have a bound graph, it will preinitialize them. But if I call “StartBehaviour” or “SwitchBehaviour” in runtime, it won’t actually preinitialize subgraphs. I would expect that if I have any settings on the graph owner, any graph that I start should behave the same way. But really, Initialize() is only called for a bound or pre-set graph and those settings work only on those. Is this an expected behaviour or maybe I am missing something?

[Gavalakis]

Hello!

This is mostly an omission in regards to use case. Here is a temporary solution for that:

1) Open up GraphOwner.cs and add Initialize call as shown below in the StartBehaviour method at line #215:

1 2 3 4 5 6 7 8 9 10 11 12

public void StartBehaviour(Graph.UpdateMode updateMode, System.Action<bool> callback = null) {             graph = GetInstance(graph);             if ( graph != null ) {                 graph.Initialize(this, blackboard, preInitializeSubGraphs); //add this call                 graph.StartGraph(this, blackboard, updateMode, callback);                 if ( onOwnerBehaviourStateChange != null ) {                     onOwnerBehaviourStateChange(this);                 }             }         }

2) Open up Graph.cs and change line #510, instead of being an assert to be a hasInitialized return check:

1 2 3 4

// Debug.Assert(!hasInitialized, "Graph is already initialized.");             if ( hasInitialized ) { return; }

Please note that this is a temporary solution. I will have to handle this better to avoid redundant calls.
Let me know if this works for you for now.

Thanks!

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

[slava]

Hi again. Thanks for a quick reply. Yeah, I’ve checked and it does work.

I did although changed the code to just instantiate and destroy the Owner. It does create additional garbage, but it’s just safer this way for my use case.

But thanks for help anyways!

[Author]

Posts