-
-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
Switch to the PyPI version of plex to generate lapack_lite #8377
Conversation
Note that |
a40ebcb
to
4c63fe1
Compare
This should be good to review now |
Could you add that prominently at the top of Changes look good to me. |
@@ -2,11 +2,10 @@ | |||
from __future__ import division, absolute_import, print_function | |||
|
|||
import sys, os | |||
from io import StringIO | |||
from io import BytesIO as StringIO |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we just use BytesIO
instead of StringIO
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, thought I hadn't left that in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Shame that Plex seems to be abandonware since 2009-12-06. It will probably never be made Python3 compatible. |
This gives exactly the same output as before
This now runs correctly on python 2, whereas before it did not run at all
4c63fe1
to
79988e1
Compare
It's now in there in various places |
Merged, thanks @eric-wieser |
The previous version, 1.1.4, was not even packaged with a
setup.py
, making installation harder. Now it can just be installed (manually) from PIP.This was tested with the lapack 3.0.0 source (not the debian lapack3 package as was used before), and was found to have identical output before and after this changeset.