Skip to content

Conversation

larsoner
Copy link
Contributor

Fixes:

invalid escape sequence \*

@effigies
Copy link
Member

Thanks. I've seen that popping up in a lot of projects lately. Is that the only one we have in nibabel?

@codecov
Copy link

codecov bot commented Oct 15, 2019

Codecov Report

Merging #826 into master will decrease coverage by 0.15%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #826      +/-   ##
==========================================
- Coverage    90.1%   89.94%   -0.16%     
==========================================
  Files          96       96              
  Lines       11910    11910              
  Branches     2125     2125              
==========================================
- Hits        10731    10712      -19     
- Misses        834      847      +13     
- Partials      345      351       +6
Impacted Files Coverage Δ
nibabel/openers.py 94.89% <ø> (ø) ⬆️
nibabel/pydicom_compat.py 77.27% <0%> (-22.73%) ⬇️
nibabel/environment.py 75% <0%> (-20%) ⬇️
nibabel/__init__.py 89.28% <0%> (-3.58%) ⬇️
nibabel/casting.py 84.68% <0%> (-0.86%) ⬇️
nibabel/viewers.py 95.39% <0%> (-0.66%) ⬇️
nibabel/dft.py 80.36% <0%> (-0.62%) ⬇️
nibabel/nifti1.py 91.14% <0%> (-0.31%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de44a10...dd52697. Read the comment docs.

@larsoner
Copy link
Contributor Author

Found a bunch more and pushed. This at least allows me to run MNE-Python test suite without errors (so far, at least I can collect the tests now!)

@effigies effigies changed the title BUG: Fix escape FIX: Change strings with invalid escapes to raw strings Oct 16, 2019
@effigies effigies merged commit debf173 into nipy:master Oct 16, 2019
@larsoner larsoner deleted the esc branch October 16, 2019 00:19
effigies added a commit that referenced this pull request Oct 16, 2019
@effigies effigies added this to the 3.0.0 milestone Oct 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants