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

[voice_text] Support new ReadSpeaker API and load libraries dynamically #395

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

[voice_text] Support new ReadSpeaker API and load libraries dynamically #395

wants to merge 8 commits into from

Conversation

mqcmd196
Copy link
Member

@mqcmd196 mqcmd196 commented Nov 8, 2022 •
edited
Loading

  • The feature for using new ReadSpeaker API.
  • Previously, rebuilding the package was required to use VoiceText, but with this PR we don't have to do it because of loading library dynamically. (also .cpp.in and dummy codes were removed)

TODO

Test with robots which use VoiceText SDK

  • Fetch
  • PR2

Test with robots which use ReadSpeaker API

  • Spot (with old network interface...?)

@mqcmd196 mqcmd196 mentioned this pull request Nov 8, 2022
Closed
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
voice_text
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mqcmd196 @k-okada