Skip to content

SSHDataGrabber outputs now returns full path to the grabbed files #1086

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

Merged
merged 3 commits into from
Apr 20, 2015

Conversation

demianw
Copy link
Contributor

@demianw demianw commented Apr 20, 2015

Corrected an issue in which the path to the downloaded files was not returned in full. Hence the pipelines were not working

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 70.48% when pulling da5adb3 on demianw:SSHDataGrabberFullPath into c1a5ea0 on nipy:master.

@satra
Copy link
Member

satra commented Apr 20, 2015

@demianw - could you please update the CHANGES file?

@demianw
Copy link
Contributor Author

demianw commented Apr 20, 2015

@satra done!

@@ -50,6 +50,7 @@ Next release
* ENH: ANTs JointFusion() (https://github.com/nipy/nipype/pull/1042)
* ENH: Added csvReader() utility (https://github.com/nipy/nipype/pull/1044)
* FIX: typo in nipype.interfaces.freesurfer.utils.py Tkregister2 (https://github.com/nipy/nipype/pull/1083)
* FIX: SSHDataGrabber outputs now return full path to the grabbed/downloaded files.
Copy link
Member

Choose a reason for hiding this comment

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

add URL : #1086

we are trying to create a direct link back from the changes so people can see the exact changes.

@demianw
Copy link
Contributor Author

demianw commented Apr 20, 2015

Great idea! Done

@satra
Copy link
Member

satra commented Apr 20, 2015

push :)

@demianw
Copy link
Contributor Author

demianw commented Apr 20, 2015

Done ;)

satra added a commit that referenced this pull request Apr 20, 2015
SSHDataGrabber outputs now returns full path to the grabbed files
@satra satra merged commit 4d0d309 into nipy:master Apr 20, 2015
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