LCAS / limo_ros2

limo ros2 packages
2 stars 9 forks source link

Separate dev #17

Closed marc-hanheide closed 8 months ago

marc-hanheide commented 8 months ago

What type of PR is this? (check all applicable)

Description

This now does the following:

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

Please try out the new on_amd64 dev container, e.g. run

and confirm that working. Optionally, try on the real platform

[optional] Are there any post deployment tasks we need to perform?

marc-hanheide commented 8 months ago

@LeonardoGuevara and @francescodelduchetto let me know if you see any issues with this as I'd like to merge for my teaching. I think this is a substantial improvement, but I still need to test on the robots. But it compiles fine for arm64 already.

francescodelduchetto commented 8 months ago

@LeonardoGuevara and @francescodelduchetto let me know if you see any issues with this as I'd like to merge for my teaching. I think this is a substantial improvement, but I still need to test on the robots. But it compiles fine for arm64 already.

I am not using the LIMOs for RBT1001, so I don't know :-)

marc-hanheide commented 8 months ago

ah. okay. then it shouldn't worry you.

marc-hanheide commented 8 months ago

ah. okay. then it shouldn't worry you.

marc-hanheide commented 8 months ago

@LeonardoGuevara, this Zenoh stuff is also the way I propose talking to our real robots in AOC!

LeonardoGuevara commented 8 months ago

Adding @jondave to this since he is the one covering the use of LIMOs.

marc-hanheide commented 8 months ago

I'd like to merge tomorrow for my workshop on Wednesday if possible, please

jondave commented 8 months ago

I don't see any issues as we are getting the students to use the limos by remotely accessing them via SSH the same as the MSc module last semester.

cooperj commented 8 months ago

Resolves #15 :)