Skip to content

Conversation

@bjeurissen
Copy link
Member

Some users pointed out to me that it was not obvious how to obtain the corrected gradient table with dwigradcheck.

bin/dwigradcheck Outdated
cmdline.set_author('Robert E. Smith (robert.smith@florey.edu.au)')
cmdline.set_synopsis('Check the orientation of the diffusion gradient table')
cmdline.add_description('Note that if the -mask command-line option is not specified, the MRtrix3 command dwi2mask will automatically be called to '
cmdline.add_description('Note that the corrected gradient table can be output using the -export_grad_{mrtrix,fsl} option. \n\n'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think if you need to have several paragraphs in the description, the right way to do it is to provide each paragraph within its own cmdline.add_description() call - not explicitly hard-code the break as two newlines.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry... Thanks for fixing!

@jdtournier jdtournier changed the base branch from dev to master July 7, 2021 09:34
@jdtournier jdtournier changed the base branch from master to dev July 7, 2021 09:54
bjeurissen and others added 3 commits July 7, 2021 10:57
Some users pointed out to me that it was not obvious how to obtain the corrected gradient table with dwigradcheck.
@jdtournier jdtournier changed the base branch from dev to master July 7, 2021 10:07
@jdtournier jdtournier enabled auto-merge July 7, 2021 10:08
@jdtournier jdtournier merged commit dc491d0 into master Jul 7, 2021
@jdtournier jdtournier deleted the dwigradcheck-doc branch July 7, 2021 10:55
@jdtournier jdtournier added this to the 3.0.3 hotfix milestone Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants