Skip to content

Question: Are the claims made on SMACC about BT valid for this implementation? #318

Closed

Description

@Levi-Armstrong

The SMACC Comparison makes claims about orthogonality and concurrency, but looking through this repos tutorials it seems those claims may not be valid for this repos implementation.

Activity

  1. SteveMacenski commented on Nov 16, 2021

    @SteveMacenski
    Contributor

    Assuming my understand of orthogonality in the SM context is correct, we've been using orthogonality in Nav2's behavior trees using BT.CPP since day 1. We have many leaf clients that call independent task servers doing independent things on independent hardware (at times), which is the same thing that SMACC does and is described in their definitions document. I can't speak to concurrancy, but I see no reason a control flow node couldn't be made to do so, but I'm not sure if BT.CPP has such a BT node example -- but I don't think there's anything preventing it.

    That entire article is incredibly biased, inaccurate, and overtly aggressive.

    A poll I ran on LinkedIn (195 responses) show a distinctly different outlook.
    image

  2. facontidavide commented on Nov 16, 2021

    @facontidavide
    Collaborator

    I am glad you asked. I agree with Steve's comment about the article being "biased, inaccurate, and overtly aggressive".

    This does not mean that BT are better than SMACC (or vice-versa).

    I am not taking any stance on this matter, because I haven't had the time, patience or need to learn SMACC.

    I expressed my opinion about that article here: https://discourse.ros.org/t/introducing-the-smacc-state-machine-library/14963/9?u=facontidavide

    The most important point, for me, is:

    we should understand which features are "implementation dependent" and which are an intrinsic property of Behavior Trees

    This is a critical distinction, because an implementation can always be improved, and this is precisely what that article fails to acknowledge.

  3. miccol commented on Nov 17, 2021

    @miccol
    Contributor

    hi @Levi-Armstrong,

    Thank you for bringing this up.
    I agree with Steve and Davide about the nature of that article.

    Like Davide, I tried to start a discussion with the author of that article here with no success.

  4. Levi-Armstrong commented on Nov 17, 2021

    @Levi-Armstrong
    Author

    Everyone, thank you for the immediate response and information which confirmed my suspicion. I will leave it up to the maintainer to close since you may prefer to leave it open.

  5. facontidavide commented on Nov 18, 2021

    @facontidavide
    Collaborator

    image

    image

    I think that keeping the strikethrough text there, instead of actually replacing it, can tell us a lot about the sincerity of those corrections and the will of the author to engage in a constructive conversation.

  6. locked and limited conversation to collaborators on Jul 21, 2023
  7. converted this issue into a discussion #613 on Jul 21, 2023
  8. unpinned this issue on Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions