nimroddolev / chime_tts

A custom Home Assistant integration to play combined audio files before and/or after text-to-speech (TTS) messages
https://nimroddolev.github.io/chime_tts/
MIT License
196 stars 14 forks source link

Error after Updating to 2024.9.0b1 #183

Closed def1149 closed 2 months ago

def1149 commented 2 months ago

System Health details

System Information

version core-2024.9.0b1
installation_type Home Assistant OS
dev false
hassio true
docker true
user root
virtualenv false
python_version 3.12.4
os_name Linux
os_version 6.6.31-haos-raspi
arch aarch64
timezone America/Chicago
config_dir /config
Home Assistant Community Store GitHub API | ok -- | -- GitHub Content | ok GitHub Web | ok HACS Data | ok GitHub API Calls Remaining | 5000 Installed Version | 2.0.0 Stage | running Available Repositories | 1389 Downloaded Repositories | 32
AccuWeather can_reach_server | ok -- | -- remaining_requests | 46
Home Assistant Cloud logged_in | true -- | -- subscription_expiration | October 2, 2024 at 7:00 PM relayer_connected | true relayer_region | us-east-1 remote_enabled | true remote_connected | true alexa_enabled | false google_enabled | false remote_server | us-east-1-9.ui.nabu.casa certificate_status | ready instance_id | 6db92b2b0db14ce798c80f31b021c265 can_reach_cert_server | ok can_reach_cloud_auth | ok can_reach_cloud | ok
Home Assistant Supervisor host_os | Home Assistant OS 13.1 -- | -- update_channel | beta supervisor_version | supervisor-2024.08.0 agent_version | 1.6.0 docker_version | 26.1.4 disk_total | 229.2 GB disk_used | 28.0 GB healthy | true supported | true host_connectivity | true supervisor_connectivity | true ntp_synchronized | true virtualization | board | yellow supervisor_api | ok version_api | ok installed_addons | Advanced SSH & Web Terminal (18.0.0), Home Assistant Google Drive Backup (0.112.1), Z-Wave JS (0.6.2), Network UPS Tools (0.13.1), SQLite Web (4.2.0), Piper (1.5.2), AWNET to HASS (1.1.2), Mosquitto broker (6.4.1), MQTT Explorer (browser-1.0.3), Frigate Proxy (1.5), Glances (0.21.1), ESPHome (2024.8.1), File editor (5.8.0), Duck DNS (1.18.0), Studio Code Server (5.15.0), Node-RED (18.0.5)
keymaster zwave_integration | zwave_js -- | -- network_status | on
Dashboards dashboards | 9 -- | -- resources | 21 views | 19 mode | storage
Recorder oldest_recorder_run | August 20, 2024 at 4:11 AM -- | -- current_recorder_run | August 29, 2024 at 12:09 PM estimated_db_size | 644.23 MiB database_engine | sqlite database_version | 3.45.3

Checklist

Describe the issue

Errors in HA Log

Reproduction steps

Update to 2024.9.0b1

Debug logs

Logger: py.warnings
Source: custom_components/chime_tts/__init__.py:9
integration: Chime TTS (documentation, issues)
First occurred: 12:09:18 PM (4 occurrences)
Last logged: 12:09:18 PM

/usr/local/lib/python3.12/site-packages/pydub/utils.py:300: SyntaxWarning: invalid escape sequence '\(' m = re.match('([su]([0-9]{1,2})p?) \(([0-9]{1,2}) bit\)$', token)
/usr/local/lib/python3.12/site-packages/pydub/utils.py:301: SyntaxWarning: invalid escape sequence '\(' m2 = re.match('([su]([0-9]{1,2})p?)( \(default\))?$', token)
/usr/local/lib/python3.12/site-packages/pydub/utils.py:310: SyntaxWarning: invalid escape sequence '\(' elif re.match('(flt)p?( \(default\))?$', token):
/usr/local/lib/python3.12/site-packages/pydub/utils.py:314: SyntaxWarning: invalid escape sequence '\(' elif re.match('(dbl)p?( \(default\))?$', token):

Diagnostics dump

No response

JonLaliberte commented 2 months ago

Yep, seeing this as well.

JonLaliberte commented 2 months ago

@nimroddolev When I set the Chime Path to a default which has a space in it, for example "Bells 2" or "Ding Dong", it fails. If I set it to something without a space, it works.

My guess is this is related?

def1149 commented 2 months ago

The only Chime I use is Bells

nimroddolev commented 2 months ago

Hey @def1149 & @JonLaliberte. Sorry you've had this issue. I have tested on v2024.9.0 and v2024.9.1 and was unable to reproduce your issue. Could it be that the issue was related to an issue in the HA beta version? Can you please test and confirm for me whether the issue persists for you past the beta? If so, please include your YAML for the action call.

JonLaliberte commented 2 months ago

@nimroddolev Sorry, I didn't notice that the OP was on a beta. I was never on HA beta though (I usually don't update until at least one point release or more as I depend on a few things working).

I was on then, and now 2024.9.1.

Anyway, I tested it just now and using "Bells 2" and "Ding Dong" are working fine now without any changes. When I tested last week I was consistently having it fail with that "invalid escape sequence" error on these, but when using sounds without spaces it was working. Maybe it was all a coincidence and a HA restart in the meantime fixed something.

def1149 commented 2 months ago

No problems now .. .thanks

nimroddolev commented 2 months ago

Great, closing the issue 👍