suever / pydicom-experimental

pydicom test
0 stars 1 forks source link

Include tags Command Dictionary from PS 3.7 Table E.1 #126

Closed suever closed 9 years ago

suever commented 9 years ago

From rickard....@gmail.com on May 14, 2013 01:49:58

There is no support for reading tags from the Command Dictionary, used for DIMSE network messaging. An update to dict_2011.csv that fixes this is available in the rickardholmberg-pydicom clone, at https://code.google.com/r/rickardholmberg-pydicom/ .

Original issue: http://code.google.com/p/pydicom/issues/detail?id=126

suever commented 9 years ago

From rickard....@gmail.com on May 13, 2013 22:51:25

By the way, the changeset itself is https://code.google.com/r/rickardholmberg-pydicom/source/detail?r=a5f573dedd77466b4bca2bd74a960c0e2dd7225f for easier viewing.

suever commented 9 years ago

From darcymason@gmail.com on May 14, 2013 19:00:20

Thanks. I'll fold these in. I see that pynetdicom adds a subset of them to the pydicom dictionary, so with a small modification to its code, this could be helpful there too.

Owner: darcymason@gmail.com

suever commented 9 years ago

From rickard....@gmail.com on May 14, 2013 23:49:15

Great, that's much appreciated. I use it in my alternative DICOM network thingy, that I recently more-or-less finished and moved to https://github.com/rickardholmberg/twisteddicom/ . (Side note: It's really not that I wanted to do a competing DIMSE implementation, but since I had gotten pretty far with it by the time Patrice released his, I wanted to finish it up for closure and learning... :-)

suever commented 9 years ago

From darcymason@gmail.com on January 11, 2014 19:48:29

This issue was closed by revision a282bc078caa .

Status: Fixed