core icon indicating copy to clipboard operation
core copied to clipboard

SyntaxWarning: invalid escape sequence in Ecovacs legacy

Open edenhaus opened this issue 1 year ago • 8 comments

The problem

I get the following warnings when using the ecovacs integration, even when I don't use the legacy library:

Logger: py.warnings
Source: components/ecovacs/controller.py:20
First occurred: 09:48:50 (2 occurrences)
Last logged: 09:48:50

/usr/local/lib/python3.12/site-packages/stringcase.py:247: SyntaxWarning: invalid escape sequence '\W' return re.sub("\W+", "", string)
/usr/local/lib/python3.12/site-packages/sleekxmppfs/thirdparty/mini_dateutil.py:163: SyntaxWarning: invalid escape sequence '\s' _iso8601_parser = re.compile("""

What version of Home Assistant Core has the issue?

2024.5.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

ecovacs

Link to integration documentation on our website

https://www.home-assistant.io/integrations/ecovacs/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

edenhaus avatar Apr 30 '24 07:04 edenhaus

Hey there @overloadut, @mib1185, @augar, mind taking a look at this issue as it has been labeled with an integration (ecovacs) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of ecovacs can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign ecovacs Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


ecovacs documentation ecovacs source (message by IssueLinks)

home-assistant[bot] avatar Apr 30 '24 07:04 home-assistant[bot]

I noticed this warning in 2024.6 as well.

WARNING (ImportExecutor_0) [py.warnings] /usr/local/lib/python3.12/site-packages/stringcase.py:247: SyntaxWarning: invalid escape sequence '\W'
  return re.sub("\W+", "", string)

crazyelectron-io avatar Jun 24 '24 08:06 crazyelectron-io

this issue is still present in 2024.8.0b1

Logger: py.warnings
Source: components/solaredge/coordinator.py:10
First occurred: 6:07:03 PM (1 occurrences)
Last logged: 6:07:03 PM

/usr/local/lib/python3.12/site-packages/stringcase.py:247: SyntaxWarning: invalid escape sequence '\W' return re.sub("\W+", "", string)

any further information, since both of the referenced pr's are merged...?

ChristophCaina avatar Aug 02 '24 16:08 ChristophCaina

@ChristophCaina your issue is not related to the ecovacs integration, but the solar edge - as per https://github.com/home-assistant/core/issues/114750#issuecomment-2174727821 this should not happen anymore, but if it still does, please open an own issue report for this

mib1185 avatar Aug 02 '24 19:08 mib1185

Oh.... Didn't Look correct. I had an issue Open for this and was Just searching for the Log MSG.

I think it was solved, but came Back.

I will Update / search my own issue. Sorry ^^

ChristophCaina avatar Aug 02 '24 20:08 ChristophCaina

@ChristophCaina the linked comment is from your previous issue 😉

mib1185 avatar Aug 02 '24 20:08 mib1185

this is also reported with 2024.10.2

Logger: py.warnings
Source: components/ecovacs/__init__.py:3
First occurred: 11:07:34 (1 occurrences)
Last logged: 11:07:34

/usr/local/lib/python3.12/site-packages/sleekxmppfs/thirdparty/mini_dateutil.py:163: SyntaxWarning: invalid escape sequence '\s' _iso8601_parser = re.compile("""

sleveille4 avatar Oct 13 '24 09:10 sleveille4

Also seeing this as of 2024.10.2 (2024.10.0 and 2024.10.1 unknown as I skipped these releases)

Logger: py.warnings
Source: components/ecovacs/__init__.py:3
First occurred: 9:27:35 AM (1 occurrences)
Last logged: 9:27:35 AM
/usr/local/lib/python3.12/site-packages/sleekxmppfs/thirdparty/mini_dateutil.py:163: SyntaxWarning: invalid escape sequence '\s' _iso8601_parser = re.compile("""

pmannk avatar Oct 14 '24 22:10 pmannk

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.