Skip to content

Conversation

blakedewey
Copy link
Contributor

Import the pickle module, which is required for workflow export
function.

Import the pickle module, which is required for workflow export
function.
@coveralls
Copy link

coveralls commented Jul 28, 2016

Coverage Status

Coverage increased (+0.007%) to 72.332% when pulling 1d02156 on fix_export into 3ed411b on master.

@blakedewey
Copy link
Contributor Author

I think the CI failure is due to the new docker setup. Not this PR.

@blakedewey blakedewey changed the title fix import error in workflow export FIX: import error in workflow export Aug 1, 2016
@satra
Copy link
Member

satra commented Aug 1, 2016

@oesteban - can you take a look at this?

@oesteban
Copy link
Contributor

oesteban commented Aug 2, 2016

@satra sure

EDIT: yes, it is a problem with the docker images. I'm going to open an issue including this and some things about docker

@oesteban oesteban self-assigned this Aug 2, 2016
oesteban added a commit to oesteban/nipype that referenced this pull request Aug 3, 2016
@yarikoptic
Copy link
Member

clearly, this is a fix to the issue where pickle module is used but not imported... not to piss anyone off but to progress the science, I will merge the beast. I hope no hurt feelings.

@yarikoptic yarikoptic merged commit 356319b into master Aug 3, 2016
@blakedewey blakedewey deleted the fix_export branch September 8, 2016 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants