Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feat/267 planner dyn reconfig #385

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Feat/267 planner dyn reconfig #385

wants to merge 26 commits into from

Conversation

Udit8348
Copy link
Contributor

@Udit8348 Udit8348 commented Mar 29, 2021 •
edited
Loading

Closes #267 by adding dynamic reconfigurability to the planner and linear filter models. A demo of it working can be found here.

Udit8348 and others added 25 commits January 25, 2021 16:46
@Udit8348
adds dynamic reconfigure package
94bf517
@Udit8348
adds starter cfg file
7765135
@Udit8348
adds callback boilerplate code
260a074
@Udit8348
first plug-in setup (failed)
0efbf6c
@Udit8348
resolved build issues
9fc957c
@Udit8348
formatting
e454f44
@Udit8348
bicycle model pointers not working
9dc0350
@Udit8348
try mutex
4eebfc8
@Udit8348
adds dyn reconf to bicycle model
98704cf
@Udit8348
adds dyn reconf to bicycle model
1a539a8
@Udit8348
Merge branch 'feat/267-planner-dyn-reconfig' of https://github.com/Ro…
73b7576 
…boJackets/roboracing-software into feat/267-planner-dyn-reconfig
@Udit8348
adds steadds speed and steering filter dyn conf
aa2f9e6
@Udit8348
adds constants to dyn config
e83755f
@Udit8348
removes mutex variable
e770aa5
@Udit8348
fixes minimum bounds
04a228f
@Udit8348
adds first loop default value getters
5e75b81
@Udit8348
fixes static bool, pass dyn reconf by reference
5fab25b
@Udit8348
corrects variable name
6136a58
@Udit8348
start to split linear tracking dyn reconf
411c335
@Udit8348
dyn reconf for linear compiles; doesnt appear
cc452d2
@Udit8348
print debugging
dffb7df
@Udit8348
Merge branch 'master' into feat/267-planner-dyn-reconfig
dad5063
@Udit8348
formatting
30cecc3
@Udit8348
correctly adds dyn reconf instance for linear
e7f8a0d
@Udit8348
clean up
9df03d1
NicoBartholomai
NicoBartholomai reviewed May 30, 2021
View reviewed changes
Copy link
Contributor

@NicoBartholomai NicoBartholomai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good to me, just some minor changes.

rr_common/cfg/lin-tracking-reconf.cfg Outdated Show resolved Hide resolved
rr_common/cfg/planner_configuration.cfg Outdated Show resolved Hide resolved
@NicoBartholomai NicoBartholomai self-requested a review November 7, 2021 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Daniel-Martin576 Daniel-Martin576 Awaiting requested review from Daniel-Martin576

@NicoBartholomai NicoBartholomai Awaiting requested review from NicoBartholomai

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Reconfigure path planner on the fly
2 participants
@Udit8348 @NicoBartholomai