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

Change all ROS dynamic_reconfigure and parameter to consistent, lowercase #13

Closed
mdhorn opened this issue Mar 17, 2016 · 1 comment
Closed

Change all ROS dynamic_reconfigure and parameter to consistent, lowercase #13

mdhorn opened this issue Mar 17, 2016 · 1 comment
Labels
enhancement

Comments

@mdhorn
Copy link

mdhorn commented Mar 17, 2016

Currently there is a mix of lower and upper case parameter names. Request all variable are changed to lower case with underscores for consistent naming convention.

This will require changes to launch files as well.
@mdhorn mdhorn assigned mdhorn and rjingar and unassigned mdhorn Mar 17, 2016
@reaganlo reaganlo mentioned this issue Mar 30, 2016
Merged
@reaganlo
Copy link

This issue has been fixed in Release 1.0.2

nlyubova pushed a commit to nlyubova/realsense that referenced this issue Apr 25, 2016
@rjingar
Modified all parameters to lowercase for consistency
8556e4c 
This fixes Issue IntelRealSense#13
icarpis pushed a commit to icarpis/realsense that referenced this issue Sep 17, 2017
Merge pull request IntelRealSense#13 from sgutlian/dev
d5a9744 
rename person tracking output topic
@comwise comwise mentioned this issue Oct 12, 2019
Closed
YoshuaNava added a commit to YoshuaNava/realsense that referenced this issue Jun 30, 2020
@YoshuaNava
Merge pull request IntelRealSense#13 from leggedrobotics/samuelba-pat…
29aad2a 
…ch-1

Add missing eigen dependency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mdhorn @reaganlo @rjingar