[cantor] [Bug 411645] Make sagemath backend compatible with sagemath built with python3
https://bugs.kde.org/show_bug.cgi?id=411645 Christoph Feck changed: What|Removed |Added Version Fixed In|19.12.1 |19.12.0 -- You are receiving this mail because: You are watching all bug changes.
[cantor] [Bug 411645] Make sagemath backend compatible with sagemath built with python3
https://bugs.kde.org/show_bug.cgi?id=411645 Nikita Sirgienko changed: What|Removed |Added Version Fixed In||19.12.1 Status|ASSIGNED|RESOLVED Latest Commit||https://invent.kde.org/kde/ ||cantor/commit/802fb9f44fa95 ||614a8bb0872ea1b4cc485913d82 Resolution|--- |FIXED --- Comment #5 from Nikita Sirgienko --- Git commit 802fb9f44fa95614a8bb0872ea1b4cc485913d82 by Nikita Sirgienko. Committed on 05/12/2019 at 22:56. Pushed by sirgienko into branch 'release/19.12'. Add compatibility with Sage based on Python3 FIXED-IN: 19.12.1 M +2-2src/backends/sage/sagesession.cpp https://invent.kde.org/kde/cantor/commit/802fb9f44fa95614a8bb0872ea1b4cc485913d82 -- You are receiving this mail because: You are watching all bug changes.
[cantor] [Bug 411645] Make sagemath backend compatible with sagemath built with python3
https://bugs.kde.org/show_bug.cgi?id=411645 --- Comment #4 from pcpa --- Hi, Yes, it would be a good change to cantor. This change will make cantor work if sagemath is built with python3 or python2. Currently upstream is building with python2, but is finishing porting to python3. I believe currently only the latest Fedora sagemath packages are built with python3. -- You are receiving this mail because: You are watching all bug changes.
[cantor] [Bug 411645] Make sagemath backend compatible with sagemath built with python3
https://bugs.kde.org/show_bug.cgi?id=411645
--- Comment #3 from Nikita Sirgienko ---
Egh, sory for delay, my email have lost a mail about your comment.
So, with line
print("%s %s" % ('TMP_DIR', sage.misc.misc.SAGE_TMP))
all tests works fine, so you want this changes to Cantor source code, right?
--
You are receiving this mail because:
You are watching all bug changes.
[cantor] [Bug 411645] Make sagemath backend compatible with sagemath built with python3
https://bugs.kde.org/show_bug.cgi?id=411645
--- Comment #2 from pcpa ---
(In reply to Nikita Sirgienko from comment #1)
> Hi
>
> First, if you are familiar with development, then there is
> phabricator.kde.org for suggestions like this.
> Second, there is tests for Cantor sage backend, and on my machine (SageMath
> 8.7 based on python2) one of them is failed (TestSage::testPlot(),
> testsage.cpp::92).
>
> So, on this moment, I can't accept the patch, because it breaks appearing of
> plots appearing for SageMath, based on python2.
>
> But, anyway, this problem is important, so, if this patch won't be improved
> for working with Python2, I think, I just will add a option "SageMath python
> 3 compatibility".
Many thanks for checking it. I am not much familiar with phabricator.kde.org
For some reason, while working on parallel with several issues related
to the sagemath rpm package, when submitting the report I though the patch
could be simplified, but it cannot.
The proper line for two arguments would be:
print("%s %s" % ('TMP_DIR', sage.misc.misc.SAGE_TMP))
python2 would generate different output. For example:
$ python2 -c 'print("foo", "bar")'
('foo', 'bar')
$ python3 -c 'print("foo", "bar")'
foo bar
$ python2 -c 'print("%s %s" % ("foo", "bar"))'
foo bar
I was actually testing a patch with the later version, but got confused and
for some reason thought it could be simplified.
--
You are receiving this mail because:
You are watching all bug changes.
[cantor] [Bug 411645] Make sagemath backend compatible with sagemath built with python3
https://bugs.kde.org/show_bug.cgi?id=411645 Nikita Sirgienko changed: What|Removed |Added Status|REPORTED|ASSIGNED Assignee|cantor-b...@kde.org |warqu...@gmail.com CC||warqu...@gmail.com Ever confirmed|0 |1 --- Comment #1 from Nikita Sirgienko --- Hi First, if you are familiar with development, then there is phabricator.kde.org for suggestions like this. Second, there is tests for Cantor sage backend, and on my machine (SageMath 8.7 based on python2) one of them is failed (TestSage::testPlot(), testsage.cpp::92). So, on this moment, I can't accept the patch, because it breaks appearing of plots appearing for SageMath, based on python2. But, anyway, this problem is important, so, if this patch won't be improved for working with Python2, I think, I just will add a option "SageMath python 3 compatibility". -- You are receiving this mail because: You are watching all bug changes.
