Mixed

The flag name aren't needed in the comment and this is in the doc. So I
removed them.

I did all the other changes.

On Mon, Oct 20, 2014 at 12:03 PM, carriepl <notifications@github.com> wrote:

> In theano/sandbox/cuda/opt.py:
>
> > @@ -1109,10 +1109,37 @@ def local_gpu_softmax_with_bias(node):
> >  from theano.tensor.nnet import conv
> >
> >
> > -@register_opt()
> > +# Need to be registered before local_gpu_conv_legacy. Otherwise, it
> > +# will have priority over this optimization.  We want, if cudnn is
> > +# available and the GPU support it, use it.  Otherwise, the gemm
> > +# version should be used.  If the user want the legacy convolution,
> > +# they should use the Theano flag:
> > +# optimizer_excluding=local_conv_gemm.
> > +# If cudnn is available, this flag should be added:
> > +# optimizer_excluding=local_gpu_conv
>
> "Needs to be registered"
> "and the GPU supports it, to use it"
> "If the users want"
>
> Also, the description of how to use the Theano flag is a bit hard to
> parse. Maybe something along the line of :
> "If the users want to use the legacy convolution, they need to use the
> Theano flag optimizer_excluding. If cudnn is not available, only the
> optimization local_conv_gemm needs to be disabled :
> [PUT FLAG DEFINITION HERE]
> If cudnn is available, the optimization local_gpu_conv also needs to be
> disabled :
> [DEFINE FLAGS TO EXCLUTE BOTH OPTIMIZATIONS]"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Theano/Theano/pull/2192/files#r19094250>.
>