dwiintensitynorm: Fix bug introduced in #1389#1534
Merged
Conversation
Function app.mrconvertOutputOption() can only be applied when run.command() is calling mrconvert; the command-line options it exploits are not present in other binaries.
Lestropie
added a commit
that referenced
this pull request
Jan 31, 2019
- dwipreproc: - Make use of command example usage functionality to demonstrate usage of four -rpe_* options. - Give more thorough description of -align_seepi option, and update some description text that was written prior the the existence of that option. - Erase the commented text at the head of the file, which formed a pragmatic 'help page' way back before creation of the MRtrix3 Python API. - If CUDA eddy fails, write its output to a text file in the scratch directory nonetheless. - Print outputs from topup and eddy for -info verbosity and above, but additionally provide --verbose command-line options to these commands if running script at -debug level. - foreach: - Break up string substitutions across individual description paragraphs. - Use example usage functionality to demonstrate basic command-line usage of command without triggering a sphinx warning. - Also add second example usage to demonstrate multi-threading. - dwinormalise group: - Add description paragraph describing necessity to use .mif format DWI files as gradient tables must be embedded (echoes #1414). - Propagate bug fix #1534. - mrtrix3.path: Provide quote() function for placing quotation marks around filesystem path if necessary as part of a command string. - dwi2response: For msmt_5tt and dhollander algorithms, use path.quote() to support use cases where the scratch directory location contains whitespace.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in 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.
Function
app.mrconvertOutputOption()can only be applied whenrun.command()is callingmrconvert; the command-line options it exploits are not present in other binaries.