Bug in "sympy.physics.quantum.cg"

20 views
Skip to first unread message

shiny...@gmail.com

unread,
Aug 19, 2015, 11:50:46 AM8/19/15
to sympy
I found a wrong expression to indicate a variable in "Wigner6j.doit()".

Original in Sympy version 0.7.6:
    def doit(self, **hints):
        if self.is_symbolic:
            raise ValueError("Coefficients must be numerical")
        return wigner_6j(self.j1, self.j2, self.j12, self.j3, self.j, self.j3)

Revised:
    def doit(self, **hints):
        if self.is_symbolic:
            raise ValueError("Coefficients must be numerical")
        return wigner_6j(self.j1, self.j2, self.j12, self.j3, self.j, self.j23)

This bug may be a simple typo...

Sean Vig

unread,
Aug 19, 2015, 12:41:58 PM8/19/15
to sy...@googlegroups.com
Hi,

Yes, that is a typo, if you'd like to submit a PR on github we'd be happy to merge it. Thanks for finding it.

Sean

--
You received this message because you are subscribed to the Google Groups "sympy" group.
To unsubscribe from this group and stop receiving emails from it, send an email to sympy+un...@googlegroups.com.
To post to this group, send email to sy...@googlegroups.com.
Visit this group at http://groups.google.com/group/sympy.
To view this discussion on the web visit https://groups.google.com/d/msgid/sympy/c4b2a5ef-4259-4519-972d-988945dbfae5%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply all
Reply to author
Forward
0 new messages