Thursday, February 09, 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: VMCCNY
New Today New Today: 18
New Yesterday New Yesterday: 26
User Count Overall: 20581

People Online People Online:
Visitors Visitors: 10
Members Members: 4
Total Total: 14

Online Now Online Now:
01: tservice
02: Chimarides
03: Leighann
04: lecalos

ClearCanvas Community Forums

Bug in C-Move-RQ at study level?
Last Post 2009-10-02 07:58 AM by LuukO. 7 Replies.
Printer Friendly
Sort:
PrevPrev NextNext
You are not authorized to post a reply.
Author Messages
LuukO
Basic Member
Basic Member
Posts:19

--
2008-11-26 09:51 AM  

Hi All,

According to the DICOM standard a C-MOVE-RQ on study level should provide a value for the Unique Key of the higher patient level (Patient ID) (See PS3.4, c.4.2.2.1).

In Clearcanvas only the StudyInstanceUID can be given:

    /// MoveScuBase moveScu = new StudyRootMoveScu("myClientAeTitle", "myServerAe", "127.0.0.1", 5678, "destinationAE");
    /// moveScu.AddStudyInstanceUid("1.3.46.670589.5.2.10.2156913941.892665384.993397");
    /// moveScu.Move();

I don't think many PACS-systems have a problem with this implementation, but in my opinion this is not completely correct implemented.

With kind regards,

Luuk

stewart
Senior Member
Senior Member
Posts:2032

--
2008-11-26 10:44 AM  
Hi Luuk, you are correct when talking about the "Patient Root" Q/R Information Model, however for the "Study Root" model, the unique key is the StudyInstanceUid.

Regards,
Stewart
Real-time support available to Clinical Edition and Team Edition customers
dblanchard
Senior Member
Senior Member
Posts:185

--
2008-11-26 10:51 AM  
Luuk does have a point, we could push the AddStudyInstanceUid() up to the StudyRootMoveScu instead of the abstract MoveScuBase. And same for the other methods, but then, we'd be repeating methods (AddStudyInstanceUid could be there for both Patient Root or Study Root)

actually what we should probably do is make a public property for the DicomAttributeCollection _dicomAttributeCollection, so caller can specify any tags s/he wants to.
stewart
Senior Member
Senior Member
Posts:2032

--
2008-11-26 01:17 PM  

Good point Dan, I was talking strictly about the Q/R models, but the methods on the MoveScuBase don't entirely make sense.  Also, I only glanced quickly at the code, but it looks as though it will only work for PATIENT or STUDY level.  The reason is that the standard actually specifies that the unique key for each level above the current query level must be supplied, but if you were to call AddStudyInstanceUid followed by AddSeriesInstanceUid (for SERIES level for example), it would actually throw an exception.  And, there's no way to access the DicomAttributeCollection directly.

I've entered a ticket for this here.

-Stewart

Real-time support available to Clinical Edition and Team Edition customers
stewart
Senior Member
Senior Member
Posts:2032

--
2008-11-26 01:47 PM  
Sorry Luuk, I assume my discovery upon code inspection is what you were talking about in your post. I was confused by your code sample which used the StudyRootMoveScu ... if it were a PatientRootMoveScu, then you would be 100% correct.

Regards,
Stewart
Real-time support available to Clinical Edition and Team Edition customers
LuukO
Basic Member
Basic Member
Posts:19

--
2008-11-27 02:05 AM  

Stewart, sorry I didn't look well in the code when I copied the example from MoveSCu.cs and I didn't mention the patient root Q/R information model.

Just for your information: The comment in the Summary-part of the region StudyRootMoveScu in MoveScu.cs mentions "Patient Root Move Scu" instead of "Study Root Move Scu" (at least in my version).

Thanks.

Luuk

 

stewart
Senior Member
Senior Member
Posts:2032

--
2009-09-29 09:42 PM  

Just an FYI, this has now been fixed.

Stewart

Real-time support available to Clinical Edition and Team Edition customers
LuukO
Basic Member
Basic Member
Posts:19

--
2009-10-02 07:58 AM  
Thanks for the update!

Luuk
You are not authorized to post a reply.

Active Forums 4.1
Copyright 2011 ClearCanvas Inc.