-
-
Notifications
You must be signed in to change notification settings - Fork 452
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
the 'sage.rings.complex_mpc' optional extension is broken in sage-4.8.alpha3 and over #12223
Comments
comment:1
the faulty C code:
|
comment:2
I am putting this to blocker since the spkg builds correctly and then because the extension is broken it breaks sage -b. |
comment:3
Just wondering: why isn't this a standard package? I guess complex arithmetic in Sage would benefit from it. |
comment:4
There was some discussion a while back when David Kirkby put it in shape last. If I remember correctly the gist of it was that you could do complex arithmetic some other way in sage and that there wasn't much benefits from it. I would have to dig sage-devel to find it. |
comment:5
Attachment: 12223_mpc.patch.gz |
Author: Jeroen Demeyer |
This comment has been minimized.
This comment has been minimized.
comment:6
See #12515 to make this a standard spkg. |
Reviewer: François Bissey |
comment:7
Ok I tested the patch and it works. I am giving this a positive review. |
Merged: sage-5.0.beta5 |
The optional mpc package installs fine but when I execute sage -b to built the sage code I get
Component: packages: optional
Author: Jeroen Demeyer
Reviewer: François Bissey
Merged: sage-5.0.beta5
Issue created by migration from https://trac.sagemath.org/ticket/12223
The text was updated successfully, but these errors were encountered: