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

Jump to bottom

Clarified documentation #2650

Merged
merged 1 commit into from Mar 31, 2020
Merged

Clarified documentation #2650

merged 1 commit into from Mar 31, 2020

Conversation

jcohenadad
Copy link
Member

This PR #2647 raised a wrong usage of the function, due to a lack of documentation.

This is now fixed in this PR.

@jcohenadad jcohenadad mentioned this pull request Mar 31, 2020
Closed
@jcohenadad jcohenadad added documentation category: readthedocs, sourceforge, or SCT courses sct_dmri_concat_b0_and_dwi context: labels Mar 31, 2020
@jcohenadad jcohenadad added this to the 4.2.3 milestone Mar 31, 2020
valosekj
valosekj reviewed Mar 31, 2020
View reviewed changes
Copy link
Member

@valosekj valosekj left a comment

Choose a reason for hiding this comment

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

It is clear for me now.

@jcohenadad jcohenadad merged commit e5eee1f into master Mar 31, 2020
@jcohenadad jcohenadad deleted the jca/dmri-concat branch March 31, 2020 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@valosekj valosekj valosekj approved these changes

Assignees
No one assigned
Labels
documentation category: readthedocs, sourceforge, or SCT courses sct_dmri_concat_b0_and_dwi context:
Projects
None yet
Milestone
4.3
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jcohenadad @valosekj