search

fossasia / susi_linux

Hardware for SUSI AI https://susi.ai
Apache License 2.0
1.61k stars 148 forks source link

make more sound outputs configurable #491

Closed norbusan closed 5 years ago

norbusan commented 5 years ago

The path to recognition_error and problem sounds are hard-coded, make them a configuration option in config.json

norbusan commented 5 years ago

I have closed this pull request. I need to think more how we can configuration make more CWD independent.

WIth the current PR this works ok, but as soon as one uses the configure.py or config_generator.py app the original set up is destroyed.

I would like to be able to have the config.json in a directory other than the susi_linux root, and thus need to make sure that the above scripts also work under these circumstances.

I will send a new PR when I know how to do it.

hongquan commented 5 years ago

Would be nice if we use .toml instead of .json.

norbusan commented 5 years ago

??? You mean for config.json? But that would need a big change, since practically all programs use json_config ;-)

But please, look at the fixed PR I sent, not this one ...