websocket server: small fix #333

Merged
mrq merged 2 commits from ben_mkiv/ai-voice-cloning:master into master 2023-08-22 19:26:42 +07:00

Hi,

this is a small fix to the websocket server implementation, which replaces "None" strings with None values in the received JSON data.

The reason for this is that the ai-voice-cloning backend sometimes wont handle "None" strings, but it would write them to the generate.json template, which renderes the WebUI unusable since it fails with a parsing error from the generate.json file

The best solution would probably be a sanity check when writing/parsing the generate.json, but i don't want to mess things up in that part of code

Hi, this is a small fix to the websocket server implementation, which replaces "None" strings with None values in the received JSON data. The reason for this is that the ai-voice-cloning backend sometimes wont handle "None" strings, but it would write them to the `generate.json` template, which renderes the WebUI unusable since it fails with a parsing error from the `generate.json` file The best solution would probably be a sanity check when writing/parsing the `generate.json`, but i don't want to mess things up in that part of code
ben_mkiv added 2 commits 2023-08-22 19:06:40 +07:00
mrq merged commit fb1cfd059f into master 2023-08-22 19:26:41 +07:00

Pull request successfully merged and closed

Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: mrq/ai-voice-cloning#333
There is no content yet.