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.

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

Add respeaker_ros package #152

Merged
merged 2 commits into from Aug 29, 2018
Merged

Add respeaker_ros package #152

merged 2 commits into from Aug 29, 2018

Conversation

furushchev
Copy link
Member

@furushchev furushchev commented Aug 22, 2018

Add a package that provides a driver for Respeaker v2 multi microphone array device.
Please see the README for more detail.
https://github.com/furushchev/jsk_3rdparty/blob/2d6cadb38aca49e8e3307fdd1f6e8aca5b488572/respeaker_ros/README.md

ros/rosdistro#18923 is required

@k-okada k-okada closed this Aug 28, 2018
@k-okada k-okada reopened this Aug 28, 2018

```bash
mkdir -p ~/catkin_ws/src && ~/catkin_ws/src
git clone https://github.com/furushchev/respeaker_ros.git
Copy link
Member

Choose a reason for hiding this comment

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

jsk-ros-pkg/respeaker_ros ?

Copy link
Member Author

Choose a reason for hiding this comment

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

@k-okada Thanks! Fixed.


## Preparation

1. Build this package
Copy link
Member

Choose a reason for hiding this comment

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

add how to use with deb package

Copy link
Member Author

Choose a reason for hiding this comment

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

@k-okada Added!

#
# To generate this file, please run:
# cd /path/to/this/package
# python ./scripts/respeaker_gencfg.py
Copy link
Member

Choose a reason for hiding this comment

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

please add when we need to run ./scripts/respeaker_gencfg.py in README.md or, may be you 'd better to use cmake to generate this file.

Copy link
Member Author

Choose a reason for hiding this comment

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

@k-okada This script is to get default value from the device as current values.
That is what I thought once before, but it requires device connected on build time.
I added an instruction to the readme.

Copy link
Member

Choose a reason for hiding this comment

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

I see, so add how to use this script and when we need to generate this file

Copy link
Member

Choose a reason for hiding this comment

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

to README.md

@k-okada k-okada merged commit 24c2bda into jsk-ros-pkg:master Aug 29, 2018
@furushchev furushchev deleted the respeaker branch September 21, 2018 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants