Skip to content

Fix/controller yaml defaults #3478

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Eldgar
Copy link

@Eldgar Eldgar commented May 24, 2025

Fix

When the user skips “Edit Selected,” Setup Assistant dropped
action_ns and default from moveit_controllers.yaml (see #3434).
This patch adds fallback values:

action_ns: follow_joint_trajectory
default: true

Test

Adds InjectedDefaultsWhenBlank to test_controllers.cpp to verify
those keys are always present.

Fixes #3434

Copy link

mergify bot commented May 24, 2025

This pull request is in conflict. Could you fix it @Eldgar?

@Eldgar Eldgar changed the base branch from main to jazzy May 24, 2025 18:05
Copy link

mergify bot commented May 24, 2025

Please target the main branch for development, we will backport the changes to jazzy for you if approved and if they don't break API.

@Eldgar Eldgar force-pushed the fix/controller_yaml_defaults branch from a354e8c to ecb1ba7 Compare May 24, 2025 18:21
Copy link

mergify bot commented May 24, 2025

This pull request is in conflict. Could you fix it @Eldgar?

@Eldgar Eldgar changed the base branch from jazzy to main May 24, 2025 18:23
@Eldgar Eldgar force-pushed the fix/controller_yaml_defaults branch from ecb1ba7 to d052d11 Compare May 24, 2025 18:29
@Eldgar Eldgar force-pushed the fix/controller_yaml_defaults branch 3 times, most recently from ce4ef51 to 6301607 Compare May 24, 2025 21:52
@Eldgar Eldgar force-pushed the fix/controller_yaml_defaults branch from 6301607 to a01de6b Compare May 24, 2025 21:54
@mikeferguson
Copy link
Contributor

I've triggered the CI - looks like the linter isn't happy with a bit of the whitespace

Copy link

codecov bot commented Jun 5, 2025

Codecov Report

Attention: Patch coverage is 95.83333% with 1 line in your changes missing coverage. Please review.

Project coverage is 46.23%. Comparing base (fb34bff) to head (14580fe).

Files with missing lines Patch % Lines
...etup_controllers/src/moveit_controllers_config.cpp 92.31% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3478      +/-   ##
==========================================
+ Coverage   46.23%   46.23%   +0.01%     
==========================================
  Files         720      720              
  Lines       62713    62738      +25     
  Branches     7591     7595       +4     
==========================================
+ Hits        28991    29003      +12     
- Misses      33555    33567      +12     
- Partials      167      168       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix joint trajectory controller duplicate key generation in setup assistant properly
2 participants