Skip to content

Refine plan demo#28

Open
charlie1329 wants to merge 23 commits intomainfrom
refine-plan-tutorial
Open

Refine plan demo#28
charlie1329 wants to merge 23 commits intomainfrom
refine-plan-tutorial

Conversation

@charlie1329
Copy link
Copy Markdown

@charlie1329 charlie1329 commented Mar 4, 2026

Introduces the REFINE-PLAN part of the overarching demo.

Here, the robot is looking for a new object (bread) in the environment. After learning from data, the robot learns to search for it anti-clockwise through the environment.

Adds a new package, refine_plan_demo, in the simulation directory. There is a README in the package on how to run it (this can be translated directly to the web page).

This PR makes the following changes to the main sim:

  • Adds new dependencies to Dockerfile/requirements.txt
  • Changes the Pyrobosim commit to support probabilistic object spawning
  • Changes the motion planner to A*. I was observing issues under RRT* where the robot was frequently taking very suboptimal paths between locations. This would render REFINE-PLAN unable to do anything useful.
  • Moved two tables slightly further apart as otherwise the navigation (regardless of planner) was unable to navigate between the two.
  • Added an override in run.py to set the detect success probability to 1.0 (we can't handle partial observability in REFINE-PLAN).

If anything isn't where you'd want it to be, let me know.

Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
… calls for navigate and detect

Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
…sn't take shortest path from hall to dining room?)

Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
… probs for the bread

Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
…ttle bit now

Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
RUN git clone -b 4.3.3 https://github.com/sea-bass/pyrobosim.git src/pyrobosim
# Using this commit instead of -b 4.3.3 because this is the first commit on main to support stochastic object spawning
RUN git clone https://github.com/sea-bass/pyrobosim.git src/pyrobosim
RUN cd src/pyrobosim; git checkout 96cff5da8e91082e955251ff0735f6b1338f5702; cd ../..
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New version 4.3.4 is out, including the probabilistic spawning required by this PR

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated!

Signed-off-by: Charlie Street <me@charliestreet.net>
Signed-off-by: Charlie Street <me@charliestreet.net>
@charlie1329
Copy link
Copy Markdown
Author

Just want to note that I've just significantly expanded the README for the REFINE-PLAN tutorial, so that'll need to be updated if you've already started the rst file in the overview repo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants