feat!: text-to-speech x LLM integration#936
Conversation
| await this.nativeModule.stream( | ||
| speed, | ||
| stopAutomatically, | ||
| (audio: number[]) => { |
There was a problem hiding this comment.
I think we should return JSTensorViewOut from the native side, so there's less copying going on
846b177 to
49b5656
Compare
|
Get errors in tests: |
|
All the stories generated in the example app are almost identical, do we want it this way? |
Well, to be honest, I was just creating those sample apps as a simple to use testing tools, since it's quite difficult to write proper tests for text-to-speech. We could remove this particular screen if we are sure that everything works fine.
Fixed. I forgot to change a no longer existing CMake variable. |
Let's keep it, I think it's better to keep it, even if very simple |
2837d23 to
f623cd6
Compare
|
FYI, I pushed one change: // CommonModelTest is not instantiated in this translation unit
namespace rnexecutorch::model_tests {
GTEST_ALLOW_UNINSTANTIATED_PARAMETERIZED_TEST(CommonModelTest);
} // namespace rnexecutorch::model_testsThis is because otherwise one test failed: and the And Claude confirms this: |
5314404 to
5bfed1c
Compare
chmjkb
left a comment
There was a problem hiding this comment.
I think the code looks good, I'm not able to run the demo apps now though.
5bfed1c to
7fa5ea4
Compare
Description
This pull request introduces a few changes to the Text-to-Speech module:
Introduces a breaking change?
Type of change
Tested on
Testing instructions
To test the Text-to-Speech module, run the set of tests for this module.
To test the new streaming mode and it's integration with text generation models, one can use 'text-to-speech-llm' demo app.
Screenshots
Related issues
#773
#897
Checklist
Additional notes