-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
unable to compute vector norms using custom numeric type #521
Comments
ofloveandhate
added a commit
to ofloveandhate/eigenpy
that referenced
this issue
Dec 18, 2024
much of the functions I added were me trying to get Python to crash in a specific way. I failed, which is a good thing, because it means that EigenPy doesn't have the bug I thought it does. BUT. EigenPy *does* have two issues exercised in the unit tests for the custom type, issues stack-of-tasks#519 and stack-of-tasks#520 . Additionally, this code exercises issue stack-of-tasks#521 , where I try to compute vector norms in two different ways and fail. Additionally, I bumped the C++ standard to C++14, since Boost 1.87 didn't work correctly with only C++11, and 1.87 is now distributed by homebrew (I develop on a Mac)
I've solved the issue in #525 |
For this part of the issue, the error is coming from NumPy directly. Their strategy does not apply for custom types. |
Solved via #525 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For the custom numeric type (mpfr complex, mpfr reals) I've implement in my pull request for the example #370 , I am unable to easily compute norms of vectors or matrices.
fails with error message
This sounds un-solvable, since I cannot convert my multiprecision complex numbers to float64.
As an alternative to using
np.norm
, I switched toThis fails with error message
That is, EigenPy doesn't provide the
square
ufunc, but it should.(I still need to push the changes in my custom numeric type example, and I've added pytest unit tests for the example which help probe failing functionality)
The text was updated successfully, but these errors were encountered: