> +            below_eye = zeros(n+m,n)
> +            A = 
> (self.mass_matrix).col_join((self._m_cd).row_join(zeros(m,m)))
> +            below_I = below_eye.row_join(A)
> +            return I.col_join(below_I)
> +        else:
> +           A = row1.col_join(row2)
> +           return A
> +
> +    @property
> +    def forcing(self):
> +        # Returns the forcing vector
> +        if self.eom == None:
> +            raise ValueError('Need to compute the equations of motion first')
> +
> +        qdd = self._qdoubledots
> +        qddzero = dict(zip(qdd, [0] * len(qdd)))

Yes, it seems to me as well, that PEP8 says you should have spaces around "*". 
In either case, I would use whatever looks more readable on the case by case 
basis, in this PR it seems that having spaces is more readable.

---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1460/files#r1328227

-- 
You received this message because you are subscribed to the Google Groups 
"sympy-patches" group.
To post to this group, send email to sympy-patches@googlegroups.com.
To unsubscribe from this group, send email to 
sympy-patches+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to