1185
93%

Ran 03 Jan 2017 07:42AM UTC

Jobs 1

Files 171

Run time 8s

Badge

Embed ▾

pending completion

Build # 1185

Build Type

Pull #501

travis-ci

Committed by

web-flow

Commit Message

Update get_csa_header to return none on CSAReadError

Some non-MRI dicom datasets have just strings in CSA Header tags. This causes the read on line 68 to throw an error because the string is not a CSA header. Unfortunately this throws out other packages such as dcmstack which use nibabel. The proposed fix returns none if a read error is encountered when such strings are parsed. It is possible such strings may pass the initial check for 0 < n_tags <=128 which throws the CSAReadError but fail later in which case the except block can be be modified to handle those exceptions as well.

Pull Request Pull Request #501: Update get_csa_header to return none on CSAReadError

Run Details

20957 of 21832 relevant lines covered (95.99%)

0.96 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
9	100.0		/home/travis/build/nipy/nibabel/venv/lib/python2.7/site-packages/nibabel/nicom/csareader.py

Jobs

ID	Job ID	Ran	Files	Coverage
4	1185.4 (COVERAGE=1)	03 Jan 2017 07:42AM UTC	0	95.99	Travis Job 1185.4

nipy / nibabel / 1185
93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 1185

nipy / nibabel / 1185 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 1185

nipy / nibabel / 1185
93%

README BADGES
x