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

rtt_rosparam: Make it better #5

Closed
3 of 4 tasks
jbohren opened this issue Aug 20, 2013 · 4 comments
Closed
3 of 4 tasks

rtt_rosparam: Make it better #5

jbohren opened this issue Aug 20, 2013 · 4 comments
Assignees
@jbohren

Comments

@jbohren
Copy link
Member

jbohren commented Aug 20, 2013

  • add better support for vectors
  • add better support for different namespace resolution (relative/absolute/private/component)
  • add service requester
  • create unit tests
@jbohren
Copy link
Member Author

jbohren commented Sep 5, 2013

@meyerj Here's a new from-scratch implementation (which should allow addition of other commonly-used container types later)

https://github.com/jhu-lcsr-forks/rtt_ros_integration/blob/new-rtt-rosparam/rtt_rosparam/src/rtt_rosparam_service.cpp

@ghost ghost assigned jbohren Sep 5, 2013
@jbohren jbohren mentioned this issue Sep 5, 2013
Closed
@jbohren
Copy link
Member Author

jbohren commented Sep 5, 2013

Getting this working also had this side-effect: ros/ros_comm#279

@jbohren
Copy link
Member Author

jbohren commented Sep 5, 2013

And here's a service requester: https://github.com/jhu-lcsr-forks/rtt_ros_integration/blob/new-rtt-rosparam/rtt_rosparam/include/rtt_rosparam/rosparam.h

This was referenced Sep 5, 2013
Merged
Closed
@jbohren
Copy link
Member Author

jbohren commented Oct 3, 2013

Closing this in favor of specific unit test ticket

@jbohren jbohren closed this as completed Oct 3, 2013
meyerj pushed a commit that referenced this issue Dec 10, 2013
Merge pull request #5 from meyerj/rtt_roscomm-template-refactoring
74f2a70 
rtt_roscomm: separated typekit pkg template from source file templates required to build the typekit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jbohren jbohren

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jbohren