Adding Problem 1 as discussed

[ct2034]

No description provided.

[sea-bass]

Thanks for doing this!

Two blocking comments:

1. You don't need to define two identical table categories in the metadata YAML file, as they are the same. You can just have a single table and create 2 instances in the Python file as you have done.

2. For the docker compose volume mounts to work correctly, you need to move the delib_ws_p1 folder into the src folder of the repo. With this change, I can enter the container and run colcon build, and the new package will actually be there to be build.

After making fix 2 above, I was able to build and run the example in the container!

---

I did find that the nav pose was wrong for the table, as seen here:

After digging, I found a small bug on my end and it is now fixed: sea-bass/pyrobosim#201

[sea-bass]

Also, it seems those CI jobs that were added in this PR have been queued for the last 6 hours... maybe it's just a retry, maybe something else?

[ct2034]

Also, it seems those CI jobs that were added in this PR have been queued for the last 6 hours... maybe it's just a retry, maybe something else?

This is because I had to make the repo private.

[ct2034]

• You don't need to define two identical table categories in the metadata YAML file, as they are the same. You can just have a single table and create 2 instances in the Python file as you have done.

1. The two tables have different colors ;-) But, I guess this does not make a big difference.

2. Usually in ROS repos, the packages are on the top-level. Do you think we could change the Docker workflow this way?

[sea-bass]

1. The two tables have different colors ;-) But, I guess this does not make a big difference.

Right, you can still do this! Just pass in color=(R, G B) in the constructor of each instance and it will override the category level color... or the same in your call to world.add_location(...)

2. Usually in ROS repos, the packages are on the top-level. Do you think we could change the Docker workflow this way?

We can! Just have to change the mount locations in the docker_compose.yaml file.

roscon24-workshop/docker-compose.yaml

Line 37 in 4e535f7

- ./src:/delib_ws/src/workshop_files:rw