From 89f05b5909d5c2a6d2139923d88c2e8ea8b335cb Mon Sep 17 00:00:00 2001 From: "alejandro.suarez@omron.com" Date: Thu, 6 Nov 2025 13:32:56 +0100 Subject: [PATCH] Fixed decorator image and text in BT basics --- docs/learn-the-basics/BT_basics.md | 7 +- .../images/DecoratorEnterRoom.svg | 211 ++++++++++-------- 2 files changed, 119 insertions(+), 99 deletions(-) diff --git a/docs/learn-the-basics/BT_basics.md b/docs/learn-the-basics/BT_basics.md index 5a92c69..19cd35e 100644 --- a/docs/learn-the-basics/BT_basics.md +++ b/docs/learn-the-basics/BT_basics.md @@ -122,9 +122,9 @@ this node could be either: The node __Inverter__ is a Decorator that inverts the result returned by its child; an Inverter followed by the node called -__isDoorClosed__ is, therefore, equivalent to +__isDoorOpen__ is, therefore, equivalent to - "Is the door open?". + "Is the door closed?". The node __Retry__ will repeat ticking the child up to __num_attempts__ times (5 in this case) if the child returns FAILURE. @@ -137,8 +137,9 @@ __Apparently__, the branch on the left side means: But... :::caution Find the BUG! +The tree expects the door to be closed so it can attempt up to 5 times to open it. If __isDoorOpen__ returns FAILURE, we have the desired behavior. -But if it returns SUCCESS, the left branch fails, and the entire Sequence +But if it returns SUCCESS (i.e. the door is already open), the bottom-left branch fails, and the top-level Sequence is interrupted. ::: diff --git a/docs/learn-the-basics/images/DecoratorEnterRoom.svg b/docs/learn-the-basics/images/DecoratorEnterRoom.svg index 0b7564b..9752d08 100644 --- a/docs/learn-the-basics/images/DecoratorEnterRoom.svg +++ b/docs/learn-the-basics/images/DecoratorEnterRoom.svg @@ -7,7 +7,7 @@ version="1.1" viewBox="0 0 1798.08 948.96" sodipodi:docname="DecoratorEnterRoom.svg" - inkscape:version="1.1.2 (0a00cf5339, 2022-02-04)" + inkscape:version="1.3 (0e150ed6c4, 2023-07-21)" xmlns:inkscape="http://www.inkscape.org/namespaces/inkscape" xmlns:sodipodi="http://sodipodi.sourceforge.net/DTD/sodipodi-0.dtd" xmlns:xlink="http://www.w3.org/1999/xlink" @@ -23,16 +23,15 @@ inkscape:pagecheckerboard="0" inkscape:deskcolor="#d1d1d1" showgrid="false" - inkscape:zoom="0.65069409" - inkscape:cx="-120.64041" - inkscape:cy="1308.6026" - inkscape:window-width="2490" - inkscape:window-height="1376" - inkscape:window-x="1990" - inkscape:window-y="27" + inkscape:zoom="0.32534705" + inkscape:cx="1349.3284" + inkscape:cy="530.20306" + inkscape:window-width="1920" + inkscape:window-height="1009" + inkscape:window-x="1912" + inkscape:window-y="-8" inkscape:window-maximized="1" - inkscape:current-layer="text3171" - inkscape:pageshadow="2" /> + inkscape:current-layer="svg398" /> - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + +