-
Notifications
You must be signed in to change notification settings - Fork 61
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
"measurementItems" value as a string, DICOM SR tid1500 #406
Comments
@ForgottenOneNyx if a given measurement does not exist, it is definitely not a good idea to add a NaN as a measurement. You should just not add that measurement at all. This means that you cannot use a fixed structure dcmqi metadata JSON file and it will need to be customized depending on what measurements are present, which is more work at the time of generating the object, but it is the correct way to do it, I believe. It is indeed the case that the numeric value communicating the measurement has DS VR (see definition of Numeric Measurement Macro, and Numeric Value). Yes, it has to remain DS, and, from what I know about the DICOM standard, at that level it cannot be changed. It is a separate question what to do if the measurement you want to communicate is indeed a NaN. I do not have the answer to this question, but I will investigate. |
As clarified by @dclunie (see below), it is possible to communicate that a measurement is a NaN, but this will require new feature to be implemented. Response from David Clunie:
|
@fedorov In case, the DCMTK is used here somewhere: Adding support for CP-2023 is already on my to-do list. By the way, this CP is a result of a discussion that I had with Stefan Allers in the newsgroup "comp.protocols.dicom". |
Support for CP-2023 has been added to the DCMTK by this commit. |
Hello, I am new to DICOM SRs, and I was wondering if we could somehow write the
value
inmeasurementItems
as a string.Considering the example given in
sr-tid1500-ct-liver-example.json
we can't because thevalue
inmeasurementItems
corresponds to "DS" definition, in common-schema.json. Does it have to remain "DS" to adhere to the DICOM SR rules? Or can it be changed.If yes, I would like it to be of a different definition which preferably includes long strings. Do you know how I can do that? I am presently using docker to create the DICOM SRs. The idea is to add the
measurementItems
but as "NaNs" if they don't exists, which can't be done by the "DS" defintion.The text was updated successfully, but these errors were encountered: