BUG: Ensure DicomUltrasoundPlugin sets DICOM.instanceUIDs #28
Conversation
|
I will have a follow-up commit also ensuring the |
|
Related pull request: |
Also save DICOM SOP instance UID into the master node so DICOM metadata can be retrieved later from the proxy nodes if needed. Co-authored-by: Andras Lasso <lasso@queensu.ca>
| # Save DICOM SOP instance UID into the master node so DICOM metadata can be retrieved later from | ||
| # the proxy nodes if needed | ||
| masterSequenceNode = sequenceBrowserNode.GetMasterSequenceNode() | ||
| masterSequenceNode.SetAttribute('DICOM.instanceUIDs', loadedNode.GetAttribute('DICOM.instanceUIDs')) | ||
|
|
||
| volumeNode = browserNode.GetProxyNode(loadedNode) | ||
| volumeNode.SetAttribute('DICOM.instanceUIDs', loadedNode.GetAttribute('DICOM.instanceUIDs')) |
There was a problem hiding this comment.
To follow-up on the commit originally posted by @lassoan in #28 (comment):
The sequence browser node can browse many different images, so in general the DICOM instance UIDs cannot be saved into that.
Instead, the attribute need to be set on the volume node in the sequence (and those attributes are copied to the proxy node).
I am not clear which approach was implied ?
- Getting the master node
- Or getting the proxy node
There was a problem hiding this comment.
You need to set the attribute of the data node that you set in the sequence.
Content of the data node in the sequence (including the attributes) are copied to the proxy node during replay.
No description provided.