Thursday, May 17, 2012
Google Custom Search

ClearCanvas Highlights

Download our Open Source software
Watch some Videos
Get the Source
Check out our Licensing
Join our  Forums
Some Research: OICR IPP-Trials

Our Community

Membership Membership:
Latest New User Latest: JBauza
New Today New Today: 19
New Yesterday New Yesterday: 33
User Count Overall: 22559

People Online People Online:
Visitors Visitors: 12
Members Members: 0
Total Total: 12

Online Now Online Now:

ClearCanvas Community Forums

Ticket #1413 (and more) implemented
Last Post 2008-01-14 12:09 PM by steve. 1 Replies.
Printer Friendly
Sort:
PrevPrev NextNext
You are not authorized to post a reply.
Author Messages
dblanchard
Senior Member
Senior Member
Posts:185

--
2008-01-11 12:18 AM  
- Ticket #1413 - It would be useful to have a DicomReadOption? to allow bypassing of pixel data when reading, for cases when the header is just being read, and the pixel data is not required.

- added another method to the DicomAttributeCollection called GetAttribute - similar to this[] but GetAttribute does not add the attribute to the collection if it doesn't exist - we had a need for this.  Would you like this too?  (I would prefer our codebase to be the same!)

- in DicomFile.Save() method, if directory of save path does not exist, try to create (note, the Directory.CreateDirectory() method could throw an exception if something strange is going on)... don't know if you want this but I put it in my code.

Regards,

Dan
   

Attachment: 080110_DicomFile_NoLoadPixelData_+more.patch

steve
Senior Member
Senior Member
Posts:1932

--
2008-01-14 12:09 PM  
Hi Dan,

First, thanks for the contribution as always!

I've gone ahead and committed the contribution with some minor changes. You can find the changes in rev 4315 of the repository.

There were some changes I made to the new GetAttrbribute() methods. First, I updated some of the text messages in error conditions to have proper hex formatting of tags. I also updated the documentation for the GetAttribute() method, there was some wrong information in the documentation, which should be correct now. There was also a change with DicomFile.Save(). Specifically, the changes you made didn't handle the case where the Filename property of DicomFile was set to just a filename, and not an absolute path. In this case, the Directory.CreateDirectory() method was throwing an exception because the path being passed to it was an empty path. I modified your changes to handle this condition and not attempt to create the directory if the Filename property was just a filename.

Thanks again for the contribution.

Best Regards,
Steve


Real-time support available to Clinical Edition and Team Edition customers
You are not authorized to post a reply.

Active Forums 4.1
Copyright 2012 ClearCanvas Inc.