Skip to content

Fix _pocketsphinx.pyx's set_alignment example#385

Merged
dhdaines merged 1 commit intocmusphinx:masterfrom
jsalsman:patch-1
May 5, 2024
Merged

Fix _pocketsphinx.pyx's set_alignment example#385
dhdaines merged 1 commit intocmusphinx:masterfrom
jsalsman:patch-1

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