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

Fix _pocketsphinx.pyx's set_alignment example #385

Merged
merged 1 commit into from
May 5, 2024

Conversation

jsalsman
Copy link
Contributor

@jsalsman jsalsman commented May 4, 2024

make the pocketsphinx.Decoder.set_alignment example print something legible

make the pocketsphinx.Decoder.set_alignment example print something legible
@jsalsman
Copy link
Contributor Author

jsalsman commented May 4, 2024

Hi @dhdaines! I'm working through the top item on slide 5 of https://bit.ly/talknicerslides and am really digging the sub-word alignment interface in Python. I wish I had known how to do this with the C API.

@dhdaines
Copy link
Contributor

dhdaines commented May 5, 2024

Ah oops! Thanks for fixing that docstring, as you noticed, those are AlignmentEntry objects and not strings.

@dhdaines dhdaines merged commit 10c1b7b into cmusphinx:master May 5, 2024
@jsalsman jsalsman deleted the patch-1 branch May 6, 2024 00:13
@jsalsman
Copy link
Contributor Author

jsalsman commented Sep 8, 2024

@dhdaines
Copy link
Contributor

dhdaines commented Sep 9, 2024

Oh... thanks for pointing this out. There's a problem in the RTD build: https://readthedocs.org/projects/pocketsphinx/builds/25218590/

I'll fix it shortly.

@dhdaines
Copy link
Contributor

dhdaines commented Sep 9, 2024

fixed in #393

@jsalsman
Copy link
Contributor Author

@dhdaines happy to help. Please comment on cmusphinx/cmudict#37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants