Potential typo regarding DecoratorNode in Introduction to BTs #16

Activity

1. Eric-SHENNONGSHI commented on Jul 7, 2025

   

   Eric-SHENNONGSHI

   on Jul 7, 2025

   Contributor

   More actions

   This is solved in 3b8a37c , but the website is not synced with the main branch
2. sprkrd

   marked Mistake in image in the Decorators section of BT basics #58 as a duplicate of this issue on Nov 6, 2025
3. sprkrd commented on Nov 6, 2025

   

   sprkrd

   on Nov 6, 2025

   Last edited by sprkrd

   More actions

   This is solved in 3b8a37c , but the website is not synced with the main branch

   @Eric-SHENNONGSHI It's been more than 2 years since this issue was created, but I still believe there's a mistake. I think the Fallback branch at the left should be a Sequence instead, judging from the text in the "Find the bug!" box and from the fact that Fallbacks aren't introduced until the next section. The BT does not make sense with a fallback unless we remove the inverter, but the point of the section is to introduce decorators using the inverter as an example.
4. sprkrd commented on Nov 6, 2025

   

   sprkrd

   on Nov 6, 2025

   Last edited by sprkrd

   More actions

   I think I see more clearly what's happening now:

   1. Originally, IsDoorOpen was correct, and changing it to IsDoorClosed was wrong. The IsDoorClosed node should be reverted back to IsDoorOpen.
   2. I think the image should depict a Sequence instead of a Fallback. Of course, a Sequence is wrong, it should be a Fallback without the inverter before IsDoorOpen, but I think that is the point of the the "Find the bug!" box below, and of introducing the Fallback node later in the next section.
   3. It's been more than 2 years since the issue was created, and still nowadays the website reflects the status before the changes of 3b8a37c: https://www.behaviortree.dev/docs/learn-the-basics/bt_basics/