Dan, Thanks for the contribution! Sorry for the delay on this, I just got back from RSNA today. I did make some mainly cosmetic changes to the code. I did change some references to Platform.Log to use the DicomLogger.Log..() routines instead. This is mainly in anticipation of us eventually removing the dependency on ClearCanvas.Common from the assembly. Resharper also recommended some changes in some of the code, including changing some of the private methods in the class to static. I've gone ahead and done that. This caused me to have to rearrange the methods to match the regions set in the source file. I think there could be some more polishing to insert more tags into the DICOMDIR, and perhaps to insert into the DICOMDIR right away when a file is added, instead of waiting until Save() is called. Also, some of the other non-IMAGE directory records should be supported below the Series level. We can expand and clean this up at some point in the future. Otherwise, it looks great, and thanks for the contribution. Steve |