Study continuation RDSRs not handled by duplicate RDSR fix
In issue ref #114 the issue of duplicate RDSRs was dealt with on the basis that each new RDSR contained all the events of the previous RDSRs. This is the situation for the newer Siemens software that creates a new RDSR fof each event, with each one being cumulative.
However, if a study is completed, then re-opened and continued, the new RDSR (at least on the older software) only contains the details of the new events. However the StudyInstanceUID is the same. This is actually the situation originally posed in issue #114, but the #114 fix would currently delete the first part of the study and replace it with the second part and lose the initial data.
IrradiationEventUIDs are recorded in the database and they are mandatory in the template so we can assume we have them. From this, we could compare the new RDSR to the database version, and instead of just checking the series time as per #114, we could do the following:
- if
StudyInstanceUID
different, import - elif
IrradiationEventUID
in database also in file, do as per#114 - else import as new study - Accession number, patient age etc will indicate link at user level.
Comments (8)
-
reporter -
reporter Need to add tests based on continued study and generalise to the x-ray template (currently expects CT template).
-
reporter Adding tests for continuation CT RDSRs. Refs
#625,#114→ <<cset dd41685c3d50>>
-
reporter Added test for duplicate DX RDSR which fails. Refs
#625→ <<cset bfb45eaecc93>>
-
reporter Generalised for projection RDSR exams. Added debug messages. Refs
#625→ <<cset 5cf5006d2512>>
-
reporter Previously didn't allow for existing to continue importing and be more than new. No tests for this section :-( Refs
#625→ <<cset e7811d5fa026>>
-
reporter Adding
#625to changes. Refs#625Docs only commit, so [skip ci] to avoid wasting pipeline minutes.→ <<cset 9d11f33d89ea>>
-
reporter - changed status to resolved
Merged in issue625continuationRDSRimport (pull request #190)
Issue625continuationRDSRimport
Fixes
#625→ <<cset 9ac09b96f625>>
- Log in to comment
Reworked the duplicate RDSR study instance UID code so it should work for cumulative RDSRs, continuation RDSRs and duplicate RDSRs. Should also avoid issue of time() being called on a NoneType. Refs
#625,#114→ <<cset 87382c2fba58>>