[Feature request] Support for Safe Coroutine

NodeCanvas Forums General Discussion [Feature request] Support for Safe Coroutine

Viewing 4 posts - 1 through 4 (of 4 total)
  • Author
    Posts
  • #8721

    Hi there,

    I make extensive use of Safe Coroutine from Edelweiss Interactive.
    For me this (next to node canvas of course:) ) is an invaluable tool and saves me a lot of work and nasty workarounds 🙂
    It basically lets you use return values from coroutines but also wait for them to finish.
    Next to that, one of the biggest advantages is that it also handles nested coroutines !

    Would it be possible for you to add support for this package (found here : https://www.assetstore.unity3d.com/en/#!/content/17316)
    The main culprit is that it needs a derivative of MonoBehaviour.

    Thanks in advance,

    /Gerard.

    #8724
    Gavalakis
    Keymaster

    Hey,

    Safe Coroutine is indeed looking usefull 🙂
    What kind of support would you like to see?
    Are you after using Safe Coroutines within an ActionTask for example?

    If the reason you can’y currently use them is because they need to start them with a MonoBehaviour, that is easily solved by using this:
    MonoManager.current.StartSafeCoroutine(MyCoroutine());

    This is also what’s happening for normal coroutines in NC behind the scenes.

    Let me know.
    Cheers!

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

    #8723

    erhm….right :blush:

    Nothing to see here, move along 😉

    It works fine like using it as you described, thanks !

    Also wanted to let you know that I’m really happy with node canvas.
    Its so nice to work with and extend !
    I use it primarily for creating my AI with behaviour trees.

    /gerard.

    #8722
    Gavalakis
    Keymaster

    Hey,

    Thanks! I am very glad you like NC and that the SafeCoroutines are working as expected 🙂
    If you stumble into anything else let me know.

    Cheers!

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

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