aravindh-krishnamoorthy
commented
2 weeks ago Unfortunately, another instance, this time stemming from mathics/builtin/numbers/linalg.py:
$ mathics -e 'A = Array[a, {3,3}]; Eigenvalues[A.ConjugateTranspose[A]]'
[...]
File "~/.conda/envs/mathics/lib/python3.11/site-packages/Mathics3-7.0.0-py3.11.egg/mathics/builtin/numbers/linalg.py", line 171, in eval
eigenvalues = list(sympy_matrix.eigenvals().items())
^^^^^^^^^^^^^^^^^^^^^^^^
[...]
File "~/.conda/envs/mathics/lib/python3.11/site-packages/Mathics3-7.0.0-py3.11.egg/mathics/core/convert/sympy.py", line 163, in __new__
return SympyExpression(self.expr)
^^^^^^^^^
AttributeError: 'SympyExpression' object has no attribute 'expr'Full traceback
``` Traceback (most recent call last): File "~/.conda/envs/mathics/bin/mathics", line 33, inSeems like this problem needs a good, long-term solution.
rocky
mmatera
Description
When using the following on GitHub built Mathics v7.0.0, a crash is observed.
More
```python File "~/.conda/envs/mathics/bin/mathics", line 33, inHow to Reproduce
Output Given
Crash, please see above.
Expected behavior
Simplified expressions for eigenvalues; likely the result obtained in the Workarounds section below.
Your Environment
Mathics 7.0.0 on CPython 3.11.10 | packaged by conda-forge | (main, Oct 16 2024, 01:27:36) [GCC 13.3.0] using SymPy 1.12.1, mpmath 1.3.0, numpy 2.1.3, cython Not installed
OS: Windows 11 WSL
Workarounds
By changing the line (of v7.0.0 code) from: https://github.com/Mathics3/mathics-core/blob/6a0b3f04484994168cec5582a83434535d8d1d14/mathics/builtin/numbers/algebra.py#L1656
to
analogous to #238, the following output is obtained:
which is likely correct. Note that the above code is restructured a bit (and modified) in the latest commit as of now:
https://github.com/Mathics3/mathics-core/blob/c5a922709e0df1bf37ecff0b7bf88bc9b16dd492/mathics/eval/numbers/algebra/simplify.py#L85
Priority
Programmable workaround exists.
Additional context
In case the workaround is fine, I can submit a PR. Otherwise, I can also help with a good fix under instructions as I'm new to Mathics.