{"action":"created","issue":{"url":"https://api.github.com/repos/chop-dbhi/dicom-anon/issues/16","labels_url":"https://api.github.com/repos/chop-dbhi/dicom-anon/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/chop-dbhi/dicom-anon/issues/16/comments","events_url":"https://api.github.com/repos/chop-dbhi/dicom-anon/issues/16/events","html_url":"https://github.com/chop-dbhi/dicom-anon/issues/16","id":82571254,"number":16,"title":"Uncaught Exception for bad tag in dicom file.","user":{"login":"sgithens","id":361700,"avatar_url":"https://avatars.githubusercontent.com/u/361700?v=3","gravatar_id":"","url":"https://api.github.com/users/sgithens","html_url":"https://github.com/sgithens","followers_url":"https://api.github.com/users/sgithens/followers","following_url":"https://api.github.com/users/sgithens/following{/other_user}","gists_url":"https://api.github.com/users/sgithens/gists{/gist_id}","starred_url":"https://api.github.com/users/sgithens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sgithens/subscriptions","organizations_url":"https://api.github.com/users/sgithens/orgs","repos_url":"https://api.github.com/users/sgithens/repos","events_url":"https://api.github.com/users/sgithens/events{/privacy}","received_events_url":"https://api.github.com/users/sgithens/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-29T20:19:46Z","updated_at":"2015-05-29T20:34:55Z","closed_at":null,"body":"While running a large batch of mammograms today, dicom-anon blew up with the following exception. It appears that the error is in the mammogram file, but I'm wondering if we shouldn't be blowing up when it occurs.\r\n\r\nWould the appropriate action in this case be to quarantine the image?  I can try to put together a patch if you think this is the appropriate behavior. (If not, what should we be doing in this case?)\r\n\r\n```\r\nTraceback (most recent call last):\r\n  File \"dicom_anon.py\", line 915, in <module>\r\n    white_list_file=white_list_file, log_file=options.log_file, rename=options.rename,profile=options.profile, overlay=options.overlay)\r\n  File \"dicom_anon.py\", line 760, in driver\r\n    ds = anonymize(ds, white_list, org_root, profile, overlay)\r\n  File \"dicom_anon.py\", line 678, in anonymize\r\n    ds.remove_private_tags()\r\n  File \"/home/sgithens/Envs/opal/lib/python2.7/site-packages/dicom/dataset.py\", line 483, in remove_private_tags\r\n    self.walk(RemoveCallback)\r\n  File \"/home/sgithens/Envs/opal/lib/python2.7/site-packages/dicom/dataset.py\", line 595, in walk\r\n    callback(self, data_element)  # self = this Dataset\r\n  File \"/usr/local/lib/python2.7/contextlib.py\", line 35, in __exit__\r\n    self.gen.throw(type, value, traceback)\r\n  File \"/home/sgithens/Envs/opal/lib/python2.7/site-packages/dicom/tagtools.py\", line 21, in tag_in_exception\r\n    raise type(e)(err)\r\nValueError: Invalid tag (0018, 1152): invalid literal for int() with base 10: '147.3'\r\n```"},"comment":{"url":"https://api.github.com/repos/chop-dbhi/dicom-anon/issues/comments/106928309","html_url":"https://github.com/chop-dbhi/dicom-anon/issues/16#issuecomment-106928309","issue_url":"https://api.github.com/repos/chop-dbhi/dicom-anon/issues/16","id":106928309,"user":{"login":"cancan101","id":51059,"avatar_url":"https://avatars.githubusercontent.com/u/51059?v=3","gravatar_id":"","url":"https://api.github.com/users/cancan101","html_url":"https://github.com/cancan101","followers_url":"https://api.github.com/users/cancan101/followers","following_url":"https://api.github.com/users/cancan101/following{/other_user}","gists_url":"https://api.github.com/users/cancan101/gists{/gist_id}","starred_url":"https://api.github.com/users/cancan101/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cancan101/subscriptions","organizations_url":"https://api.github.com/users/cancan101/orgs","repos_url":"https://api.github.com/users/cancan101/repos","events_url":"https://api.github.com/users/cancan101/events{/privacy}","received_events_url":"https://api.github.com/users/cancan101/received_events","type":"User","site_admin":false},"created_at":"2015-05-29T20:34:55Z","updated_at":"2015-05-29T20:34:55Z","body":"It seems that most fields can just be blanked out. i.e. I think this exception is thrown when reading the value for the purposes of writing the audit db."}}