search

SeiictyUsui / pydicom

Automatically exported from code.google.com/p/pydicom
0 stars 0 forks source link

Name collision for Sequence/non-sequence #38

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
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 reported on code.google.com by darcymason@gmail.com on 13 Feb 2009 at 2:32

GoogleCodeExporter commented 9 years ago 
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.

Original comment by darcymason@gmail.com on 25 May 2009 at 2:30

GoogleCodeExporter commented 9 years ago 
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.

Original comment by darcymason@gmail.com on 17 Jan 2012 at 3:42