-
Notifications
You must be signed in to change notification settings - Fork 532
MultiProc Plugin is double-submitting jobs #705
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
Comments
Can you replicate this bug on one of our example workflows? |
Well, this makes no sense. I am fairly certain that I have narrowed down the offending commit to this one. Every commit including and after it shows the bug, and every commit I've tried before it is fine. If you haven't clicked on the link to the commit, do so now. So, that's why I'm confused. @satra, do you have any memory of this commit? Can you think of what might be going on? Sorry I have been unable to concoct a dummy workflow to show the problem, although @toddt has replicated the issue with his setup on Mindhive. |
Also has anyone successfully used the IPython plugin recently? It is totally crashing for me. |
see if this happens when you remove these lines from multiproc.py:
|
Huh. That does seem to fix it. |
I can confirm it as well. |
Current status: befuddled. |
Surely you meant perplexed. |
the only piece that code does is to pass file handles to the subprocess call. https://github.com/nipy/nipype/blob/master/nipype/interfaces/base.py#L1154 hence my real question whether these are just shared processes (check with pstree) or actually distinct processes. |
I don't have pstree on my mac but they are both R+ processes in ps with On Sat, Nov 2, 2013 at 1:54 PM, Satrajit Ghosh [email protected]:
|
so very likely the process is being forked, which results in memory duplication, which in turn is bringing your computer to a halt. replacing with alternate output mode. |
yep, it's almost certainly swapping; my hard drive was making constant On Sat, Nov 2, 2013 at 2:13 PM, Satrajit Ghosh [email protected]:
|
* master: (95 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (293 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (291 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (322 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (293 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (293 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (293 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (293 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (293 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (322 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (293 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
* master: (293 commits) fix: cleaned up file removal to pay attention to related files such as img/hdr/mat, BRIK/HEAD fix: switch to allatonce mode to prevent forking (closes nipy#705) fixed version checking sty: white spaces enh: added checkspecs to makefiel ref: removing unneeded tests ref: FILMGLS needs its own test because of api changes in FSL 5.0.5 fix: update test generator to take manual edits into account tst: added autogenerated input and output spec tests enh: modified checkspecs to write output spec tests enh: modified checkspecs to write input spec tests sty: remove commented interface fix: remove deprecation warning. doc: include travis and coveralls badges sty: pep8 and metadata fixes enh: added coveralls support Allow ArtifactDetect to correctly handle AFNI motion correction parameters generated with -dfile or -1Dfile fix: updated doctest, pep8 and unit test doc: updated doctest to use literal string fix: display cmdline string explicitly ...
This fixes the real issue behind nipy#705
This fixes the real issue behind nipy#705
This problem is summarized in this thread: https://groups.google.com/forum/#!topic/nipy-user/CPjld7rIsMk
Basically, running a workflow with MultiProc and asking for a specific
n_proc
runs (2 * n_proc) processes, with each job duplicated. I do not see similar behavior with the Linear or IPython plugins, and the behavior is occurring (for me) on an older CPU that does not do any hyperthreading fanciness. The double computation is definitely happening, as asking for a MultiProc execution with as many procs as I have cores ground my computer to a complete halt.I'll try to put together a test workflow to produce this behavior.
The text was updated successfully, but these errors were encountered: