Yes, I think that's right.


On Wed, May 8, 2013 at 10:44 AM, Frédéric Bastien
<notifications@github.com>wrote:

> In theano/gof/vm.py:
>
> > +        (These updates will be applied after computations.)
> > +
> > +        Parameters
> > +        ----------
> > +        updated_vars - dict [ input variable ] -> [ output variable ]
> > +            The update dictionary, of the sort passed to theano.function
> > +
> > +        check_correctness - bool
> > +            If the linker has already "accepted" an fgraph, then there are
> > +            some quick correctness checks that can be done, namely, that the
> > +            values of `updated_vars` are in fact outputs, and they are at the
> > +            end of the fgraph outputs list, and they are in the no_recycling
> > +            list.
> > +
> > +        N.B. Not all VMs in this file actually implement the updates, things
> > +        are in transition. VMs that do not implement the updates should leave
>
> Just to be sure, there is only the Stack VM that need to be changed? I
> just want to know I follow correctly.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Theano/Theano/pull/1308/files#r4135323>
> .
>