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

Default ScannerID should be NULL instead of 0 in mri_protocol when a protocol is valid on all scanners #637

Closed
cmadjar opened this issue Jun 11, 2021 · 0 comments

Comments

@cmadjar
Copy link
Collaborator

cmadjar commented Jun 11, 2021

Modify the perl pipeline to reflect the change in the schema for the mri_protocol table. From now on, when a protocol should not be restricted to a specific scanner, instead of using 0 as a value, the field will be set to NULL.

Unless I am mistaken, I believe that most of the code that will need to be changed is located in the following function of MRI.pm:

sub identify_scan_db {

This goes hand in hand with the following issue on the LORIS side: aces/Loris#7476

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants