Weasis uses PatientPseudoUID to compare different patients #26
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Weasis uses PatientPseudoUID to compare different patients. This new method creates a new way to compare these patients. You can use a property (patientComparator.buildPatientPseudoUID) to use just the fields you want. For example: If you want to create a PatientPseudoUID just with name and patientID, you add the following property -> patientComparator.buildPatientPseudoUID=patientId,patientName (divided by comma) However, if you prefer, you do not need to use this property, and you can keep using the normal way to create the PatientPseudoUID.
This new method creates a new way to compare these patients. You can use a property (patientComparator.buildPatientPseudoUID) to use just the fields you want. For example: If you want to create a PatientPseudoUID just with name and patientID, you add the following property -> patientComparator.buildPatientPseudoUID=patientId,patientName (divided by comma) However, if you prefer, you do not need to use this property, and you can keep using the normal way to create the PatientPseudoUID.
# Conflicts: # weasis-dicom/weasis-dicom-codec/src/main/java/org/weasis/dicom/codec/utils/DicomMediaUtils.java # weasis-dicom/weasis-dicom-codec/src/main/java/org/weasis/dicom/codec/utils/PatientComparator.java
The field ID was removed; Modification in buildPatientPseudoUID;
|
Be careful with PatientPseudoUID because this UID is a key in the DICOM model (cannot be changed or updated). There is now a merge strategy in some cases, see f346618 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This new method creates a new way to compare these patients.
You can use a property (patientComparator.buildPatientPseudoUID) to use
just the fields you want.
For example:
If you want to create a PatientPseudoUID just with name and patientID,
you add the following property -> patientComparator.buildPatientPseudoUID=patientId,patientName
(divided by comma)
However, if you prefer, you do not need to use this property,
and you can keep using the normal way to create the PatientPseudoUID.
--
About the Java modification:
I created a new class that center the Patient comparation;
I deleted the old buildPatientPseudoUID in DicomMediaUtils;
And I modified all the old calls - from DicomMediaUtils.buildPatientPseudoUID to PatientComparator.buildPatientPseudoUID