opendr-eu / opendr

A modular, open and non-proprietary toolkit for core robotic functionalities by harnessing deep learning
Apache License 2.0
615 stars 95 forks source link

Merge `develop` into `master` #389

Closed ad-daniel closed 1 year ago

ad-daniel commented 1 year ago

No conflicts. I've also adapted the branch arguments (of docker files) to point to master in this branch

tsampazk commented 1 year ago

Maybe we can make a last minute addition on the changelog regarding ROS2 as it is a sizeable addition? I entirely forgot about it in the relevant PR, sorry. Also, maybe a general entry about the several bugfixes in the ROS1 nodes?

ad-daniel commented 1 year ago

Yes definitely a good idea, can't at the moment, may I ask you to?

ad-daniel commented 1 year ago

I've quickly gone through it again and seems fine. After merging this, we can trigger the new packages/docker images and quickly check those work as intended as well

tsampazk commented 1 year ago

Thanks Daniel, once again you beat me to it! :laughing: