Ok, if you do just go ahead and push to master. I rebased and pushed this
PR.


On Sun, Nov 10, 2013 at 12:12 PM, Trevor Bekolay
<notifications@github.com>wrote:

> In nengo_ocl/ast_conversion.py:
>
> > @@ -418,9 +418,9 @@ class OCL_Translator(ast.NodeVisitor):
> >      builtins = __builtin__.__dict__
> >
> >      def _check_vector_length(self, length):
> > -        if length > self.MAX_VECTOR_LENGTH:
> > -            raise ValueError("Vectors of length >%s are not supported"
> > -                             % self.MAX_VECTOR_LENGTH)
> > +        assert length > self.MAX_VECTOR_LENGTH, (
> > +            "Vectors of length >%s are not supported"
> > +            % self.MAX_VECTOR_LENGTH)
>
> Okay; I'll change sim_ocl.plan_SimDirect to catch the ValueError instead
> then.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ctn-waterloo/nengo_ocl/pull/49/files#r7549244>
> .
>