Yeah, that's alright.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy/pull/1416#issuecomment-7649009
--
You received this message because you are subscribed to the Google Groups
sympy-patches group.
To post to this group, send email to
elif ask(Q.real(expr.exp), assumptions):
-if ask(Q.positive(expr.base), assumptions):
+if expr.exp.is_Rational and \
It's this if statement that isn't tested. You'll need some tests that look at
``x**(p/q)`` for different assumptions.
Also, this
+ from sympy import S, sqrt
+ from sympy.physics.quantum.dagger import Dagger
+ from sympy.physics.quantum.spin import JzKet
+ from sympy.physics.quantum.density import Density, fidelity
+ from sympy.physics.quantum.represent import represent
+
+ up =
+ from sympy.physics.quantum.spin import JzKet
+ from sympy.physics.quantum.density import Density, fidelity
+ from sympy.physics.quantum.represent import represent
+
+ up = JzKet(S(1)/2,S(1)/2)
+ down = JzKet(S(1)/2,-S(1)/2)
+ amp = 1/sqrt(2)
+ updown =
+ updown2 = (sqrt(3)/2 )* up + (1/2)*down
+
+ print fidelity(up_dm, up_dm)
+1
+ print fidelity(up_dm, down_dm) #orthogonal states
+0
+ print fidelity(up_dm, updown_dm).evalf().round(3)
+0.707
+
+
+state1 = represent(state1) if isinstance(state1,
+ print fidelity(up_dm, down_dm) #orthogonal states
+0
+ print fidelity(up_dm, updown_dm).evalf().round(3)
+0.707
+
+
+state1 = represent(state1) if isinstance(state1, Density) else state1
+state2 = represent(state2) if isinstance(state2, Density) else state2
+
This should be fixed since #115 was merged.
---
Reply to this email directly or view it on GitHub:
https://github.com/sympy/sympy-bot/issues/105#issuecomment-7614759
--
You received this message because you are subscribed to the Google Groups
sympy-patches group.
To post to this group, send
Move processing of bool args for review command so it is not encountered when
running the list command.
closes #122
You can merge this Pull Request by running:
git pull https://github.com/flacjacket/sympy-bot list_fix
Or you can view, comment on it, or merge it online at:
This doesn't necessarily fix the problems with internationalization, but it
does silence the warnings that come up in the normal doc building and it
doesn't make the i18n docs any worse (previously, none of the translations
would show up at the bottom of the page, tho it tried to reference the
Currently, this is still kind of hacked together and mostly up for review, so
any feedback/ways to improve things/how you would like to see this sort of
thing work is much appreciated.
This primarily does two things:
- It allows you to specify multiple interpreters by adding multiple ``-i``
10 matches
Mail list logo