Download NIfTI file from imaging browser when a NIfTI file is available in addition to a MINC file #8206
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…TI file is available in addition to the MINC file
cmadjar
added
the
24.0.0-bugs
|
@nicolasbrossard could you please review/test this? Thanks! |
nicolasbrossard
approved these changes
Dec 8, 2022
| @@ -826,6 +826,10 @@ class ImageDownloadButtons extends Component { | |||
| BaseURL={this.props.BaseURL} | |||
| Label="Download NRRD" | |||
| /> | |||
| <DownloadButton FileName={this.props.NiiFile} | |||
| BaseURL={this.props.BaseURL} | |||
| Label="Download NIfTI" | |||
There was a problem hiding this comment.
I did not know that NIfTI was spelt with a lowercase "f"...Noted!
|
@nicolasbrossard thank you for the review. The documentation is pretty limited but I modified the HELP text file |
nicolasbrossard
approved these changes
Dec 8, 2022
|
@driusan ready for you! |
driusan
reviewed
Dec 9, 2022
| @@ -1,6 +1,6 @@ | |||
| # Imaging Browser: View Session | |||
|
|
|||
| This page displays a candidate's scans collected from a single timepoint. Click on any image to launch the BrainBrowser visualization tool. With every file displayed, there are different actions you can take, such as change the file's QC status, add or edit the QC comments, download the MINC file, or see a longitudinal view (in BrainBrowser). Click **Header Info** to see further scan parameters from the DICOM header. | |||
| This page displays a candidate's scans collected from a single timepoint. Click on any image to launch the BrainBrowser visualization tool. With every file displayed, there are different actions you can take, such as change the file's QC status, add or edit the QC comments, download the imaging files (MINC/NIfTI), or see a longitudinal view (in BrainBrowser). Click **Header Info** to see further scan parameters from the DICOM header. | |||
There was a problem hiding this comment.
I find the "with every file displayed" wording confusing but it was there before so it's beyond the scope of this PR..
zaliqarosli
pushed a commit
to zaliqarosli/Loris
that referenced
this pull request
Mar 6, 2023
…TI file is available in addition to a MINC file (aces#8206) This modifies the imaging browser so that NIfTI files associated to a MINC file can be downloaded through the imaging browser view session page. Currently, we can download BVAL and BVEC files when they are present for DWI but could not download the NIfTI file that comes with it. This is a bug fix for projects using MINC files and producing NIfTI files at the time of insertion so that they can download.
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Brief summary of changes
This modifies the imaging browser so that NIfTI files associated to a MINC file can be downloaded through the imaging browser view session page. Currently, we can download BVAL and BVEC files when they are present for DWI but could not download the NIfTI file that comes with it.
This is a bug fix for projects using MINC files and producing NIfTI files at the time of insertion so that they can download.
Testing instructions (if applicable)
Download Image=> will download a MINC file (extension.mnc)Download BVAL=> will download a .bval text fileDownload BVEC=> will download a .bvec text fileDownload NIfTIfor raisinbread DWI MINC files with NIfTI files