search

suever / pydicom-experimental

pydicom test
0 stars 1 forks source link

Name collision for Sequence/non-sequence #38

Closed suever closed 9 years ago

suever commented 9 years ago

From darcymason@gmail.com on February 12, 2009 21:32:37

See r95 , r96 . Temp fix put in for "Other Patient IDs" colliding with "Other Patient IDs Sequence" when CleanName drops the "Sequence". Should make a more general solution that "Sequence" is kept for any name collisions like this.

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

suever commented 9 years ago

From darcymason@gmail.com on May 24, 2009 19:30:15

Could consider whether to have name "...Sequence" for all sequences, use the English plural only as a short-form. However, I do like the usual python philosophy of "only one right way to do it", so maybe could deprecate the plural short-form and remove eventually. Has the advantage of simplifying the code and the documentation.

Owner: ---
Labels: Difficulty-Medium

suever commented 9 years ago

From darcymason@gmail.com on January 16, 2012 19:42:48

Shift to new DICOM keywords made in recent revision. Deprecation of old plural names will be put in after move to python 2.6 as minimum supported python version.

Status: Fixed