Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update get_csa_header to return none on CSAReadError #501

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Commits on Jan 3, 2017

  1. 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.
    parneshraniga authored Jan 3, 2017
    Configuration menu
    Copy the full SHA
    f83ed47 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    4a73b36 View commit details
    Browse the repository at this point in the history

Commits on Jan 4, 2017

  1. Configuration menu
    Copy the full SHA
    920cf49 View commit details
    Browse the repository at this point in the history

Commits on Jan 12, 2017

  1. Merge remote-tracking branch 'upstream/master'

    Raniga, Parnesh (H&B, Herston - RBWH) authored and Raniga, Parnesh (H&B, Herston - RBWH) committed Jan 12, 2017
    Configuration menu
    Copy the full SHA
    2c1d332 View commit details
    Browse the repository at this point in the history