Thanks for having a look over it. Wasn't sure how you wanted your code
structured, so I'll go ahead and make the changes you requested. We've had
a few more days to test the SLURM feature and I'm going to make a few
usability changes as well before the next pull request.


On Tue, Aug 13, 2013 at 12:25 PM, Satrajit Ghosh
<notifications@github.com>wrote:

> In nipype/pipeline/plugins/base.py:
>
> > +    """
> > +
> > +    def __init__(self, template, plugin_args=None):
> > +        super(SLURMLikeBatchManagerBase, self).__init__(template=template, plugin_args=plugin_args)
> > +        self._template = template
> > +        self._sbatch_args = None
> > +        if plugin_args:
> > +            if 'template' in plugin_args:
> > +                self._template = plugin_args['template']
> > +                if os.path.isfile(self._template):
> > +                    self._template = open(self._template).read()
> > +            if 'sbatch_args' in plugin_args:
> > +                self._sbatch_args = plugin_args['sbatch_args']
> > +        self._pending = {}
> > +
> > +    def _get_result(self, taskid):
>
> if you are concerned about the mit hack, if it doesn't apply, it won't add
> a performance penalty.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nipy/nipype/pull/629/files#r5742231>
> .
>