search

cmusphinx / sphinxtrain

Acoustic model trainer for CMU Sphinx
Other
178 stars 112 forks source link

Add includes and fix number of params for bio call #21

Closed lenzo-ka closed 3 years ago

lenzo-ka commented 3 years ago

This fixes function references that had the wrong number of arguments (missing swap), and adds includes for implicitly defined functions. With this, it builds again.

Depends on https://github.com/cmusphinx/sphinxbase/pull/82

nshmyrev commented 3 years ago

Thank you Kevin. If it is really you, let me know if you want developer rights here.

lenzo-ka commented 3 years ago

Hi —

Thanks! Yes, it’s me. I’m at Duolingo now, so I can contribute much more easily than when I was at Apple.

Best regards,

Kevin

On Sep 24, 2020, at 5:46 PM, Nickolay V. Shmyrev notifications@github.com wrote:

 Thank you Kevin. If it is really you, let me know if you want developer rights here.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

lenzo-ka commented 3 years ago

Oh and yes it would be nice to have that access!

Thanks again,

Kevin

On Sep 24, 2020, at 6:00 PM, Kevin Lenzo lenzo@duolingo.com wrote:

 Hi —

Thanks! Yes, it’s me. I’m at Duolingo now, so I can contribute much more easily than when I was at Apple.

Best regards,

Kevin

On Sep 24, 2020, at 5:46 PM, Nickolay V. Shmyrev notifications@github.com wrote:

 Thank you Kevin. If it is really you, let me know if you want developer rights here.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

nshmyrev commented 3 years ago

I sent the invite, thank you!