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

[mri_violations] Can not resolve violations if scan has multiple violations #7472

Closed
driusan opened this issue Jun 10, 2021 · 2 comments
Closed
Assignees
Labels
Category: Bug PR or issue that aims to report or fix a bug

Comments

@driusan
Copy link
Collaborator

driusan commented Jun 10, 2021

When there are multiple violations for a scan in the mri_violations module, they can not be resolved. The page reloads without changing the data.

The hash for identifying the violation in the _process function is built as md5(concat_WS( ':',MincFile,PatientName,SeriesUID,TimeRun)). This is used in a pselectOne while trying to get the primary key for the resolution table. However, the values concatenated are not enough to uniquely identify a row if a scan has multiple violations, resulting in the pselectOne failing because of the multiple rows returned.

@driusan driusan added the Category: Bug PR or issue that aims to report or fix a bug label Jun 10, 2021
@ridz1208
Copy link
Collaborator

might be solved by #7473

@driusan
Copy link
Collaborator Author

driusan commented Feb 21, 2023

I'm willing to cross my fingers and hope you're right that that PR fixed it.

@driusan driusan closed this as completed Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Bug PR or issue that aims to report or fix a bug
Projects
None yet
Development

No branches or pull requests

2 participants