[Help needed] Enable search by modality in _PATIENT_ROOT and _STUDY_ROOT

[EmilioAHV]

I tried this approach, changing the attributes in pynetdicom.apps.qrscp.db file:

_PATIENT_ROOT_ATTRIBUTES = OrderedDict(
    {
        "PATIENT": ["PatientID", "PatientName","Modality"],
        "STUDY": [
            "StudyInstanceUID",
            "StudyDate",
            "StudyTime",
            "AccessionNumber",
            "StudyID","Modality"
        ],
        "SERIES": ["SeriesInstanceUID", "Modality", "SeriesNumber"],
        "IMAGE": ["SOPInstanceUID", "InstanceNumber"],
    }
)
_STUDY_ROOT_ATTRIBUTES = OrderedDict(
    {
        "STUDY": [
            "StudyInstanceUID",
            "StudyDate",
            "StudyTime",
            "AccessionNumber",
            "StudyID","PatientID",
            "PatientName",
            "Modality"
        ],
        "SERIES": ["SeriesInstanceUID", "Modality", "SeriesNumber"],
        "IMAGE": ["SOPInstanceUID", "InstanceNumber"],
    }
)

It's not working so I was hoping to recieve some feedback,

[scaramallion]

You should already be able to do so at the SERIES level? Could you provide more context on what you're trying to achieve?

[EmilioAHV]

Sure, I was running some tests with the QRSCP app, using Weasis (i have this viewer connected to a local Orthanc instance, where i C-GET some images and then upload them to QRSCP) which works just fine. But when i do a C-FIND according to the image below:

[snip]

All the instances appears (if i add more parameters to the query criteria works well) but when I do the same in orthanc (C-FIND only by modality, in this case DX) it works (also appears additional info such as series description and modality.

[snip]

This also happens with Radiant (when C-FIND with only modality, all the studies appears but modality and series description doesnt).
This what the viewers send to QRSCP:
I: # DICOM Dataset
I: (0008,0020) DA (no value available) # 0 StudyDate
I: (0008,0030) TM (no value available) # 0 StudyTime
I: (0008,0050) SH (no value available) # 0 AccessionNumber
I: (0008,0052) CS [STUDY] # 1 QueryRetrieveLevel
I: (0008,0061) CS [DX] # 1 ModalitiesInStudy
I: (0008,0090) PN (no value available) # 0 ReferringPhysicianName
I: (0008,1030) LO (no value available) # 0 StudyDescription
I: (0010,0010) PN (no value available) # 0 PatientName
I: (0010,0020) LO (no value available) # 0 PatientID
I: (0010,0021) LO (no value available) # 0 IssuerOfPatientID
I: (0010,0030) DA (no value available) # 0 PatientBirthDate
I: (0010,0040) CS (no value available) # 0 PatientSex
I: (0020,000D) UI (no value available) # 0 StudyInstanceUID
I: (0020,0010) SH (no value available) # 0 StudyID
I: (0020,1206) IS (no value available) # 0 NumberOfStudyRelatedSeries
I: (0020,1208) IS (no value available) # 0 NumberOfStudyRelatedInstances

[scaramallion]

Please don't upload screenshots with patient data

[EmilioAHV]

Oh, srry for that, they were uploaded on weasis as demos. I thought they were random tests.
About the issue with C-FIND what could be the problem? I have confirmed that in the sql all the medata from patient is properly loaded