Added verified Dockerised workflows' instructions for running Interactive Web & Nav2 Integration demos - #48
Conversation
… of the box for docker users. Signed-off-by: Gary Bey <beyhy94@gmail.com>
…er build approach. Signed-off-by: Gary Bey <beyhy94@gmail.com>
…yle of previous section header. Signed-off-by: Gary Bey <beyhy94@gmail.com>
…prototype with linked sub-README.md. Signed-off-by: Gary Bey <beyhy94@gmail.com>
… README.md with reference to Dockerfile.nav2. Signed-off-by: Gary Bey <beyhy94@gmail.com>
…ADME.md. Signed-off-by: Gary Bey <beyhy94@gmail.com>
…vailable for containerized display. Signed-off-by: Gary Bey <beyhy94@gmail.com>
| @@ -0,0 +1,37 @@ | |||
| ARG ROS_DISTRO=jazzy | |||
|
|
|||
There was a problem hiding this comment.
Thanks for this docker file! It makes sense to create a separate image for nav2 robots, however right now this docker file will essentially have exactly the same deps as our custom ros2_rust_base development image. This makes me wonder what value this current docker file adds. I do think packaging the nice project nicely is very important so we should think carefully about what "artifacts" we'd like exported.
(PS Id also recommend adding a CI workflow that actually builds any dockerfile that we produce otherwise it'll lead to random floating dockerfiles that are broken/untested later on down the line).
There was a problem hiding this comment.
Ah noted.
The original intent of this Dockerfile.nav2 is to automate the manual installation steps in Manual Setup of Nav2 Demo README.md so users need not have to do so.
Let me run the nav2 simulation with Dockerfile.base file shortly. If the simulation still runs as expected, I would probably have Dockerfile.nav2 removed in favour of less random floating dockerfiles.
Will also look into adding the CI workflow in a seperate PR. Will link here in the thread once available.
There was a problem hiding this comment.
Progress Update 💬
Managed to get nav2 integration demo working with just Dockerfile.base and a few proposed changes.
Will remove Dockerfile.nav2 for now.
Signed-off-by: Gary Bey <beyhy94@gmail.com>
…stead of Dockerfile.nav2. Signed-off-by: Gary Bey <beyhy94@gmail.com>
… colcon-building workspace. Signed-off-by: Gary Bey <beyhy94@gmail.com>
…tions in wall of text. Signed-off-by: Gary Bey <beyhy94@gmail.com>
…blish example in nav2 integration package README.md. Signed-off-by: Gary Bey <beyhy94@gmail.com>
Signed-off-by: Gary Bey <beyhy94@gmail.com>
…lt-in switch to zenoh and cyclone DDS. Signed-off-by: Gary Bey <beyhy94@gmail.com>
New Feature Implementation 🌟
Implemented Feature(s) 📚
README.mdandnav2_integration/README.md.Dockerfile.nav2base image file underdocker/which includes the manual steps defined inSetupsection ofnav2_integration/README.mdREADME.mdin rootREADME.mdto make it more accessible and apparent for new users.Implementation Description 💬
The docker-centric instructions complement the existing 2 build and run approaches, namely
localanddistrobox, in rootREADME.md.It is mainly catered to users who are more familiar with using just docker and to make explicit environment and configurations needed to allow GUI display for Nav2 Integration demonstration.
Note
Here's how the documentation looks like:
GenAI Use
We follow OSRA's policy on GenAI tools