Potential typo regarding DecoratorNode in Introduction to BTs

[mamariomiamo]

Hello,

For this section, "An Inverter followed by the node called isDoorOpen is...", this seems to be inconsistent with the node shown in the given figure which has the node labelled "isDoorClosed".

Do correct me if I understand it wrongly.

Thank you very much.

[Eric-SHENNONGSHI]

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

[sprkrd]

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.

[sprkrd]

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/