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

Cast to string to get correctly ns from rosparam in voxel evaluation #2016

Merged
merged 1 commit into from
Feb 9, 2017
Merged

Cast to string to get correctly ns from rosparam in voxel evaluation #2016

merged 1 commit into from
Feb 9, 2017

Conversation

wkentaro
Copy link
Member

@wkentaro wkentaro commented Feb 8, 2017 •
edited
Loading

Problem

When we set ~marker_ns:=2, it is interpreted as int, but namespace must be string.
That's why I need this change.

@wkentaro wkentaro changed the title Cast to string to get correctly ns from rosparam Cast to string to get correctly ns from rosparam in voxel evaluation Feb 8, 2017
@wkentaro wkentaro requested a review from iory February 9, 2017 05:46
@wkentaro wkentaro added bug and removed enhancement labels Feb 9, 2017
@k-okada k-okada merged commit 52d86d8 into jsk-ros-pkg:master Feb 9, 2017
@wkentaro wkentaro deleted the cast_to_string_marker_ns branch February 9, 2017 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iory iory Awaiting requested review from iory

@furushchev furushchev Awaiting requested review from furushchev

Assignees
No one assigned
Labels
bug pkg/jsk_pcl_ros_utils
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wkentaro @k-okada