Guard node does not work in nested trees

Forums 💬 NodeCanvas ⚙️ Support Guard node does not work in nested trees

Viewing 8 posts - 1 through 8 (of 8 total)
  • Author
    Posts
  • #17327
    w
    Participant

      Guard node does not seem to be working in sub-trees. OnGraphStarted raises a null-reference error since graphAgent is null in subgraphs. Not sure if this is intended behavior but shouldn’t the agent get propagated down to subtrees upon subgraph initialization? If this is not a bug, is there an alternative solution?

      Thanks

      public override void OnGraphStarted(){
      SetGuards(graphAgent);
      }

      #17334
      Gavalakis
      Keymaster

        Hello,

        Thanks for the report.
        The agent is actually propagated down the subGraph, but the error is happening, because of the current execution order in “SubTree.CheckInstance” method (OnGraphStarted is called before propagating the agent).
        I will have to fix this.
        Thanks again for the report!

        #17333
        w
        Participant

          Thank you. Looking forward to the fix.

          #17332
          w
          Participant

            Hi Gavalakis,

            Just following up on this bug. can we expect a fix for it any time soon?

            #17331
            Gavalakis
            Keymaster

              Hey,

              I expect to send a new version (which will include this fix) to the asset store within the next week 🙂
              Thank you.

              #17330
              w
              Participant

                Awesome! Thanks.

                #17329
                Gavalakis
                Keymaster

                  Hello again,

                  I have attached for you here a quick fix so that you don’t have to wait for next version release.
                  Let me know if you encounter any problems with this.

                  Thank you!

                  #17328
                  w
                  Participant

                    Thanks a million!

                  Viewing 8 posts - 1 through 8 (of 8 total)
                  • You must be logged in to reply to this topic.