Loris icon indicating copy to clipboard operation
Loris copied to clipboard

Add phase encoding direction and echo number to mri protocol and mri protocol violated scans 24.1 release

Open cmadjar opened this issue 3 years ago • 1 comments

Brief summary of changes

This adds the columns PhaseEncodingDirections and EchoNumber to the mri_protocol and mri_protocol_violated_scans tables so that those headers could be used to discriminate between some MRI sequences.

Testing instructions (if applicable)

  1. Test the SQL patch
  2. Test the default schema on an empty DB

Link(s) to related issue(s)

  • Did not create an issue. This was discussed during a roadmap call.

Related to PR on LORIS-MRI side: https://github.com/aces/Loris-MRI/pull/782

cmadjar avatar Aug 10 '22 13:08 cmadjar

@driusan eventually, we will need to modify the mri_violations module to display those new fields. I see that image_type is not in the mri_violations either so I was thinking of sending a different PR to fix this so that at least the MRI side of things work for the insertion.

Thoughts? If you agree, then I will create an issue and assign to me. Thanks!

cmadjar avatar Aug 10 '22 13:08 cmadjar