-
Notifications
You must be signed in to change notification settings - Fork 51
Feat/267 planner dyn reconfig #385
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
Udit8348
wants to merge
26
commits into
master
Choose a base branch
from
feat/267-planner-dyn-reconfig
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 25 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
94bf517
adds dynamic reconfigure package
Udit8348 7765135
adds starter cfg file
Udit8348 260a074
adds callback boilerplate code
Udit8348 0efbf6c
first plug-in setup (failed)
Udit8348 9fc957c
resolved build issues
Udit8348 e454f44
formatting
Udit8348 9dc0350
bicycle model pointers not working
Udit8348 4eebfc8
try mutex
Udit8348 98704cf
adds dyn reconf to bicycle model
Udit8348 1a539a8
adds dyn reconf to bicycle model
73b7576
Merge branch 'feat/267-planner-dyn-reconfig' of https://github.com/Ro…
Udit8348 aa2f9e6
adds steadds speed and steering filter dyn conf
Udit8348 e83755f
adds constants to dyn config
Udit8348 e770aa5
removes mutex variable
Udit8348 04a228f
fixes minimum bounds
Udit8348 5e75b81
adds first loop default value getters
Udit8348 5fab25b
fixes static bool, pass dyn reconf by reference
Udit8348 6136a58
corrects variable name
Udit8348 411c335
start to split linear tracking dyn reconf
Udit8348 cc452d2
dyn reconf for linear compiles; doesnt appear
Udit8348 dffb7df
print debugging
Udit8348 dad5063
Merge branch 'master' into feat/267-planner-dyn-reconfig
Udit8348 30cecc3
formatting
Udit8348 e7f8a0d
correctly adds dyn reconf instance for linear
Udit8348 9df03d1
clean up
Udit8348 22a8131
Apply suggestions from code review
NicoBartholomai File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| #!/usr/bin/env python | ||
| PACKAGE = "rr_common" | ||
|
|
||
| from dynamic_reconfigure.parameter_generator_catkin import * | ||
|
|
||
| gen = ParameterGenerator() | ||
|
|
||
| # Linear Tracking Filter Varibles (arbitrary max and min values) | ||
| gen.add("val_max", double_t, 0, "linear model: maximum value", 0, 0.0, 30.0) | ||
| gen.add("val_min", double_t, 0, "linear model: minimum value", 0, -5.0, 0.0) | ||
| gen.add("rate_max", double_t, 0, "linear model: maximum rate", 0, 0.0, 50.0) | ||
| gen.add("rate_min", double_t, 0, "linear model: minimum rate", 0, -50.0, 0.0) | ||
|
|
||
|
|
||
| exit(gen.generate(PACKAGE, "rr_common", "LinearTracking")) | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,26 @@ | ||
| #!/usr/bin/env python | ||
| PACKAGE = "rr_common" | ||
|
|
||
| from dynamic_reconfigure.parameter_generator_catkin import * | ||
|
|
||
| gen = ParameterGenerator() | ||
|
|
||
| # Planner Params | ||
| # Must be 10 or less provided the matrix size | ||
| gen.add("n_segments", int_t, 0, "planner: segment size", 7, 1, 10) | ||
|
|
||
| # Bicycle Model Params | ||
| gen.add("max_lateral_accel", double_t, 0, "bicycle model: max lateral acceleration", 7.0, 0.0, 14.0) | ||
| gen.add("segment_size", int_t, 0, "bicycle model: segment size", 25, 1, 50) | ||
| gen.add("dt", double_t, 0, "bicycle model: dt", 0.02, 0.001, 2.0) | ||
|
|
||
| # Control Constants | ||
| gen.add("steering_gain", double_t, 0, "steering gain", 1.4, 0.00, 3.0) | ||
| gen.add("k_map_cost", double_t, 0, "cost: map constant", 0.1, 0.00, 1.0) | ||
| gen.add("k_global_path_cost", double_t, 0, "cost: global map cost", 0.0, -10.0, 100) | ||
| gen.add("k_speed", double_t, 0, "cost: speed constant", 0.05, 0.00, 1.0) | ||
| gen.add("k_steering", double_t, 0, "cost: steering constant", 0.01, 0.00, 1.0) | ||
| gen.add("k_angle", double_t, 0, "cost: angle constant", 0.00, 0.00, 1.0) | ||
| gen.add("collision_penalty", double_t, 0, "cost: collision penalty", 10000, 100, 20000) | ||
|
|
||
| exit(gen.generate(PACKAGE, "rr_common", "PathPlanner")) | ||
|
NicoBartholomai marked this conversation as resolved.
Outdated
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.