#20842 assigned defect

Modeller and seqalign invalid escape sequences

Reported by: Tom Goddard Owned by: Eric Pettersen
Priority: moderate Milestone:
Component: Core Version:
Keywords: Cc:
Blocked By: Blocking:
Notify when closed: Platform: all
Project: ChimeraX

Description

The ChimeraX test suite is complaining in nightly builds about a couple instances of invalid escape sequences in strings like \. and \s that will "not work in the future". Not sure how they won't work, maybe they mean it will be a hard syntax error when importing the file. At any rate it would be good to change them to the correct
. and
s so the test suite output is not littered.

tests/test_imports_app.py::test_imports[chimerax.modeller.common]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/modeller/common.py:607: SyntaxWarning: "\." is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
."? A raw string is also an option.

generated_model_pattern = re.compile('.*\.B.*\.pdb') # aka *.B*.pdb

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:47: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'MSF:\s*(\S*)\s*' # length?

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:48: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'Type:\s*(\S*)\s*' # type

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:49: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'(.*)\s*' # date/time

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:50: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'Check:\s*(\S*)\s*' # checksum

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:51: SyntaxWarning: "\." is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
."? A raw string is also an option.

'\.\.') # signature

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:53: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'MSF:\s*(\S*)\s*' # length?

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:56: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'Check:\s*(\S*)\s*' # checksum

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:57: SyntaxWarning: "\." is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
."? A raw string is also an option.

'\.\.') # signature

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:60: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'Len:\s*(\S*)\s*' # length

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:61: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'Check:\s*(\S*)\s*' # checksum

tests/test_imports_app.py::test_imports[chimerax.seqalign.io.readMSF]

/Users/runner/work/ChimeraX/ChimeraX/chimerax/seqalign/io/readMSF.py:62: SyntaxWarning: "\s" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "
s"? A raw string is also an option.

'Weight:\s*(\S*)\s*') # weight

Change History (0)

Note: See TracTickets for help on using tickets.