Skip to content

[FIX] address dm_qc_report edge case + db update issues#333

Merged
DESm1th merged 3 commits intoTIGRLab:masterfrom
DESm1th:qc_fixes
Aug 29, 2022
Merged

[FIX] address dm_qc_report edge case + db update issues#333
DESm1th merged 3 commits intoTIGRLab:masterfrom
DESm1th:qc_fixes

Conversation

@DESm1th
Copy link
Copy Markdown
Contributor

@DESm1th DESm1th commented Jul 20, 2022

This PR addresses three bugs in dm_qc_report.py

  • Phantoms with anat data were resubmitting to the queue every night, because their manifest file contained only an empty dict, leading to the file write being skipped. This led these phantoms to appear as though they had not completely generated. (2861e75)
  • Scan lengths were not being added to the database because a relative path was being given to update_dashboard and the files couldnt be found. The error was also being missed in the logs, because the log level was debug. (0e195e3)
  • Header diffs werent updating when a new gold standard is found, because the new standard was not being provided to the function call. This led to some sessions appearing as though they had not fully generated and repeatedly resubmitting to the queue. (8c8317d)

@DESm1th DESm1th merged commit b4d9cce into TIGRLab:master Aug 29, 2022
@DESm1th DESm1th deleted the qc_fixes branch August 29, 2022 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant