[bug] Timeout task doesn't reset properly

Forums 💬 NodeCanvas ⚙️ Support [bug] Timeout task doesn't reset properly

Viewing 5 posts - 1 through 5 (of 5 total)
  • Author
    Posts
  • #17219
    w
    Participant

      The timer coroutine inside the timeout task doesn’t stop upon reset(). this causes problems when stop/restart graphs.

      #17223
      Gavalakis
      Keymaster

        Hello,

        The “Timeout” ConditionTask is better only to only be used in the context of FSMs.
        If you are referring to BehaviourTrees, it would be far better to use the “Timeout” Decorator node instead.
        Are you indeed referring to using the “Timeout” ConditionTask in BehaviourTrees?

        Let me know.
        Thanks!

        #17222
        w
        Participant

          Hi Gavalakis,

          Thank you for your explanation. I indeed was referring to using the timeout condition task in BT. Didn’t know it was designed for FSM. That makes sense.

          What we need is an “inverted-timeout” decorator, which returns failure and blocks its child connection until a timeout. We have been achieving this by using the combination of conditional + timeout. (or wait until + timeout)

          We also sometimes use timeout on a Condition node

          #17221
          Gavalakis
          Keymaster

            Hello again,

            Thanks for the follow up. Indeed there are numerous results that can be achieved by combining different decorators.
            Another “time” related decorator that you can look into, is the “Filter” decorator, set to work as a “CoolDown”, which might or might not be what you want in your particular case of course.

            Thanks!

            #17220
            w
            Participant

              Thanks Gavalakis. CoolDown is not exactly what we need since we need the Countdown to start immediately, not after first execution. We have written our own decorator to handle this situation.

              Thank you!

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