ros2-dotnet / ros2_dotnet

.NET bindings for ROS2
Apache License 2.0
139 stars 57 forks source link

Implemented parameter handling #127

Open Chootin opened 8 months ago

Chootin commented 8 months ago

This PR adds parameter handling to nodes including parameter overrides and support for use_sim_time on the node clock.

The PR also addresses the TODO regarding reading and using RMW definitions for quality of service profiles over the interop. -> Was extracted to #131

Finally it updates the rcldotnet_examples/RCLDotnetTalker.cs example to demonstrate using a parameter.

Chootin commented 8 months ago

Happy new year @hoffmann-stefan!

I have pulled together the commits from my fork relevant to parameter handling and QoS improvements when you have some time to have a look. Cheers!

renemoll commented 7 months ago

@Chootin, first of all thanks for this patch, I am already using it.

I did have to make a fix on the QoS header, see: https://github.com/renemoll/ros2_dotnet/commit/ea1993f8a5c99477d225acc551d794c93d624fa3

Chootin commented 7 months ago

@Chootin, first of all thanks for this patch, I am already using it.

I did have to make a fix on the QoS header, see: renemoll@ea1993f

Thanks for catching that! I've merged it in.

hoffmann-stefan commented 2 months ago

Part of this was merged in #131

@Chootin: Could you force push my branch onto this? This was updated at the same time I split the Commits for #131, so you don't have to rebase everything with merge conflicts again. You can view the diff locally between your current branch and mine, there are only other changes that got merged in the meantime to main.

Something like that if you haven't done this before:

# add my remote
$ git remote add origin-hoffmann-stefan https://github.com/hoffmann-stefan/ros2_dotnet.git
$ git fetch origin-hoffmann-stefan
# Diff to check that nothing got changed
$ git diff tutina/upstream_pr2 origin-hoffmann-stefan/upstream_pr2_params_rebased
# Reset your branch to the last commit of my branch
$ git reset --hard origin-hoffmann-stefan/upstream_pr2_params_rebased
# Force push to update this PR
$ git push -f
Chootin commented 2 months ago

I have completed the requested force push - looks like it can now be merged. @hoffmann-stefan