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

fix new line for warning message #584

Merged
merged 20 commits into from
Feb 23, 2022

Conversation

NPann
Copy link
Contributor

@NPann NPann commented Feb 22, 2022

Fix stdout from

Warning: Intensity scale/slope using 0028,1053 and 0028,1052Convert 1 DICOM as /flywheel/v0/work/mDIXON_Quant_BH_901_WIP_mDIXON-Quant_BH_20180110130625_901_e2_fieldmaphz (288x288x38x2)

to

Warning: Intensity scale/slope using 0028,1053 and 0028,1052
Convert 1 DICOM as /flywheel/v0/work/mDIXON_Quant_BH_901_WIP_mDIXON-Quant_BH_20180110130625_901_e2_fieldmaphz (288x288x38x2)

which let nipype interface parse the output correctly...

@neurolabusc
Copy link
Collaborator

Thanks for this suggestion. Please submit pull requests to the 'development' branch. Alternatively, I can make this change for you. The 'master' branch refers to the current stable release, and a new stable release is generated twice per yer.

@NPann NPann changed the base branch from master to development February 23, 2022 00:18
@NPann NPann changed the base branch from development to master February 23, 2022 00:26
@NPann NPann changed the base branch from master to development February 23, 2022 00:26
@NPann
Copy link
Contributor Author

NPann commented Feb 23, 2022

@neurolabusc I updated the target branch. Sorry for this confusion.

EDIT: Actually I screwed things up it seems. I'll fix it.

@neurolabusc neurolabusc merged commit 05c5061 into rordenlab:development Feb 23, 2022
@NPann
Copy link
Contributor Author

NPann commented Feb 23, 2022

@neurolabusc I am bit confused to why there are 4 file changed in this MR. As mentioned I think I screwed up with the github online editor when resolving the conflict with development branch. Let me know your thoughts.

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.

2 participants