-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[runtime] Fixed failed compilation without ITN. Now, compiling ITN is mandatory. #2444
Conversation
不传itn的fst就可以了 |
@xingchensong 使用的时候是不传fst就可以。 |
wetext这个库体积很大吗?不把他编进去的主要好处是什么 |
不是体积大小的原因。 |
我倾向于直接改cmakelist,把itn这个可选性删掉,变成必选项。 因为即便按照现在这种方式修改了,配套自己的itn还是要改代码的。 |
按照我们目前的方案,是不需要修改wenet代码,只在外层增加代码就行,方案大概是这样的: 另外,还有个小问题,现在使用itn后,单字的时间戳是不是对应不上了… |
如果是这样的话,把itn编译进去(但是不使用),也不用魔改wenet代码 |
是可以这么做,但是必须编译的话,又感觉不够简洁了,而我又暂时没想出别的简单方案 |
thx。把wetext当成boost一样的基础库来看就不觉得编译进去不简洁了,反而placeholder的方式显式增加了代码注入 |
In the original version, it didn't work without itn. but now a further separation of references 'wetext' can ensure that it works without itn.