search

edickie / ciftify

The tools of the Human Connectome Project (HCP) adapted for working with non-HCP datasets
https://edickie.github.io/ciftify/
MIT License
116 stars 157 forks source link

Deprecation warnings due to invalid escape sequences #149

Closed tirkarthi closed 2 years ago

tirkarthi commented 3 years ago

Deprecation warnings are raised due to invalid escape sequences. This can be fixed by using raw strings or escaping the literals. pyupgrade also helps in automatic conversion : https://github.com/asottile/pyupgrade/

find . -iname '*.py' | xargs -P4 -I{} python3.9 -Wall -m py_compile {}         
  File "./tests/integration/run_CNP_recon_all.py", line 128
    sym_lab = 'no' is no_symlinks else 'sy'
                                  ^
SyntaxError: invalid syntax

./ciftify/bin/ciftify_PINT_vertices.py:159: DeprecationWarning: invalid escape sequence \ 
  logo = """
./ciftify/utils.py:132: DeprecationWarning: invalid escape sequence \ 
  logo = """
./ciftify/config.py:425: DeprecationWarning: invalid escape sequence \s
  cmd_pieces = re.split('^-|\s-', cmd_args)
./ciftify/config.py:437: DeprecationWarning: invalid escape sequence \s
  nifti_inputs = re.findall('-i\s*\S*|-T2\s*\S*', cmd_args)
DESm1th commented 2 years ago

Closing this because the issue was resolved here https://github.com/edickie/ciftify/pull/156