Qiskit
/
qiskit
/
15740289108
/
1
88%
main: 88%
main: 88%
|
Ran
|
Files
827
|
Run time
29s
|
Badge
Embed ▾
README BADGES
|
coverage: 87.981% (+0.002%) from 87.979%
push
github
web-flowFix sympify() for expressions with vector element .subs() (#14641) (#14642) This commit fixes an issue in the .sympify() method when building the Sympy expression for a `ParameterExpression` that was constructed by calling `ParameterExpression.subs()` with a ParameterVectorElement type. The previous implementation of this method was passing the symbols for substitution by string which required sympy to "parse" the input. The square brackets for indexing the vector element in it's string representation was not valid for sympy to parse and this caused the error. To avoid this problem this commit adjusts the usage to pass a sypy expression object instead of a string. This is the intent of the function as string parsing has other issues in sympy and this was an oversight in the implementation that was doing this. By passing a sympy object to sympy it is able to handle the vector's name with square brackets. Fixes #14640 (cherry picked from commit 69d7dbd3e) Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
83047 of 94392 relevant lines covered (87.98%)
516903.82 hits per line
