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.

Sign up for GitHub

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

Jump to bottom

[dataquery] dataquery module should include all data from CouchDB_Import_MRI #9374

Open
1 of 29 tasks
driusan opened this issue Sep 30, 2024 · 1 comment · May be fixed by #9545
Open
1 of 29 tasks

[dataquery] dataquery module should include all data from CouchDB_Import_MRI #9374

driusan opened this issue Sep 30, 2024 · 1 comment · May be fixed by #9545
Assignees
@nicolasbrossard
Labels
Module: dataquery PR or issue related to (new) dataquery module Module: imaging_browser PR or issue related to imaging_browser module Priority: High PR or issue should be prioritised over others for review and testing Proposal PR or issue suggesting an improvement that can be accepted, rejected or altered

Comments

@driusan
Copy link
Collaborator

driusan commented Sep 30, 2024 •
edited
Loading

This will probably need to be split up into different tickets and closed, but for now I'm making one for auditing the fields from the CouchDB Import MRI script and comparing that the fields are all in the new dataquery tool.

The fields from the script that need to be verified as present are:

  • QCComment (Session)
  • for each scan type:
    • selected
    • qcstatus
    • foreach getFeedbackMRICommentType:
      • Comment
      • foreach predefinedComments
        • PredefinedComment (Yes/No)
    • ScannerID
    • Pipeline
    • OutputType
    • AcquisitionProtocol
    • CoordinateSpace
    • Algorithm
    • AcquisitionDate
    • FileInsertDate
    • SeriesDescription
    • SeriesNumber
    • EchoTime
    • RepititionTime
    • SliceThickness
    • Time (No of volumes)
    • Comment
    • SlicewiseRejected
    • InterlaceRejected
    • IntergradientRejected
    • TotalRejected
    • Caveat
    • ProcessingPipeline
@driusan driusan added the Proposal PR or issue suggesting an improvement that can be accepted, rejected or altered label Sep 30, 2024
@driusan driusan added the Priority: High PR or issue should be prioritised over others for review and testing label Dec 12, 2024
@ridz1208
Copy link
Collaborator

example from meeting https://github.com/aces/Loris/pull/9537/files

@driusan driusan added Module: imaging_browser PR or issue related to imaging_browser module Module: dataquery PR or issue related to (new) dataquery module labels Jan 21, 2025
@nicolasbrossard nicolasbrossard linked a pull request Jan 29, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nicolasbrossard nicolasbrossard

Labels
Module: dataquery PR or issue related to (new) dataquery module Module: imaging_browser PR or issue related to imaging_browser module Priority: High PR or issue should be prioritised over others for review and testing Proposal PR or issue suggesting an improvement that can be accepted, rejected or altered
Projects
None yet
Milestone
No milestone
3 participants
@driusan @nicolasbrossard @ridz1208