Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need Some Clue for Implement Adapter of Cornerstone Rectangle Annotation #143

Open
jenhaoyang opened this issue Aug 5, 2020 · 0 comments

Comments

@jenhaoyang
Copy link

I saw these implement hints from dcmjs/src/utilities/TID300/index.js. But I am not clear about those hint, could you explain more detail?
Is that are the Hierarchy I need to implement in dcmjs/src/utilities/TID300/Rectangle.js like dcmjs/src/utilities/TID300/Ellipse.js do?
I saw "Graphic Type: POLYLINE" and I wondering Rectangle.js should change Graphic Type to POLYLINE. But what is the meaning of following line?
TID 1500 MeasurementReport
--TID 1501 Measurement Group
---Measurement Group (DCM 125007)
----TID 300 Measurement

// - Cornerstone Rectangle ROI
//
// Hierarchy
// TID 1500 MeasurementReport
// --TID 1501 Measurement Group
// ---Measurement Group (DCM 125007)
// ----TID 300 Measurement
// ------SCOORD. Graphic Type: POLYLINE
// ------ Use concept corresponding to Rectangle measurement
//
// http://dicom.nema.org/medical/dicom/current/output/html/part16.html#sect_TID_4019

The hint also mention about "This should be the same as a Freehand ROI, more or less." Is that mean I need to use src/adapters/Cornerstone/Freehand.js as my template for implement src/adapters/Cornerstone/Rectangle.js rather than src/adapters/Cornerstone/EllipticalRoi.js?

// OR
// Note: This should be the same as a Freehand ROI, more or less. We add a TID 4019: Algorithm Identification flag to specify that this was created (and should be rehydrated) into a Rectangle ROI.
// TODO: Should we use a Derivation instead? http://dicom.nema.org/medical/dicom/current/output/html/part16.html#DCM_121401
// Should specify the Area measured in mmˆ2, including the units in UCUM
// Should specify the Mean Modality Pixel Value measured in whatever units the image is in
// Should specify the Standard Deviation Modality Pixel Value measured in whatever units the image is in

Originally posted by @jenhaoyang in #137 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant