Skip to content

FIX: import error in workflow export #1552

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 1 commit into from
Aug 3, 2016
Merged

FIX: import error in workflow export #1552

merged 1 commit into from
Aug 3, 2016

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