search

awslabs / python-deequ

Python API for Deequ
Apache License 2.0
730 stars 136 forks source link

Invalid escape sequence #235

Open iWantToKeepAnon opened 1 month ago

iWantToKeepAnon commented 1 month ago

Describe the bug configs.py gives SyntaxError: invalid escape sequence '\.' from IS_DEEQU_V1 = re.search("com\.amazon\.deequ\:deequ\:1.*", DEEQU_MAVEN_COORD) is not None

Raw strings or double backslashes are needed.

To Reproduce This only happens when running pytest and not from the REPL. I have not reproduced this independently yet. But dot and colon are not valid escape sequences.

Expected behavior No errors from invalid escape sequences.

Desktop (please complete the following information): macos running 6.10.0-linuxkit docker image.

Additional context

    from pydeequ.configs import SPARK_VERSION
E     File "/root/poetry/venv/lib/python3.10/site-packages/pydeequ/configs.py", line 43
E       IS_DEEQU_V1 = re.search("com\.amazon\.deequ\:deequ\:1.*", DEEQU_MAVEN_COORD) is not None
E                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
E   SyntaxError: invalid escape sequence '\.'
iWantToKeepAnon commented 1 month ago

There is no need for a regular expression search here. This would be the same:

IS_DEEQU_V1 = "com.amazon.deequ:deequ:1" in DEEQU_MAVEN_COORD