BUG (critical): Optional decorator blocks recursive node reset

Forums 💬 NodeCanvas ⚙️ Support BUG (critical): Optional decorator blocks recursive node reset

Viewing 4 posts - 1 through 4 (of 4 total)
  • Author
    Posts
  • #18213
    bcristian
    Participant

      Hello

      Using the “optional” decorator blocks the recursive graph node reset, because unless the node below is running, it reports it as “resting”.
      But the recursive reset stops when it reaches a “resting” connection. In turn, iterators for example stop working.

      #18216
      Gavalakis
      Keymaster

        Thanks for the bug report.
        Probably the best solution to that is to add a new status. I did some tests and it works fine.
        Just open up Status.cs and add Optional at the end of the enum.

        Then open up Optional.cs and change it by the end to return:

        The node/connection status coloring will be a bit off now though. I’ve already fixed them here. If you want I can tell you how/where to do so as well.

        Thanks again

        #18215
        bcristian
        Participant

          Thanks.
          Fixing the coloring would be great, too.

          #18214
          Gavalakis
          Keymaster

            You are welcome.
            I’ve send you an email with the fixes.
            Cheers!

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