#26299 closed enhancement (fixed)
py3: use repr for floats in plot3d file format outputs
Reported by:  embray  Owned by:  

Priority:  minor  Milestone:  sage8.4 
Component:  python3  Keywords:  
Cc:  Merged in:  
Authors:  Erik Bray  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  06cce34 (Commits)  Commit:  
Dependencies:  Stopgaps: 
Description
This can be seen as a continuation of #25247.
Change History (17)
comment:1 Changed 10 months ago by
 Status changed from new to needs_review
comment:2 Changed 10 months ago by
 Status changed from needs_review to needs_work
comment:3 Changed 10 months ago by
 Commit changed from aa0a1088e9063b36ac14c31402ce1bc031ad4f03 to 24fc2d1b19a752bd5d9bb95f298e627d1aa7b2ff
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
24fc2d1  p3: use more explit float.__repr__ instead of float.__str__ for plot3d

comment:4 Changed 10 months ago by
 Status changed from needs_work to needs_review
comment:5 Changed 10 months ago by
 Commit changed from 24fc2d1b19a752bd5d9bb95f298e627d1aa7b2ff to 2cec77ca93dca18ad4e41a517560523cc866a04f
Branch pushed to git repo; I updated commit sha1. New commits:
2cec77c  py3: a few more string formatting enhancements in str > repr for floats

comment:6 Changed 10 months ago by
Added a couple more
comment:7 Changed 10 months ago by
 Status changed from needs_review to needs_work
one failing doctest
sage t long src/sage/plot/plot3d/parametric_surface.pyx
comment:8 Changed 10 months ago by
Ah...it seems that when you use fstrings in Cython on Python 2 it returns unicode
strings, somewhat annoyingly (but also reasonably).
comment:9 Changed 10 months ago by
 Commit changed from 2cec77ca93dca18ad4e41a517560523cc866a04f to 06cce349f9c8bee1236241fbb53c02202b7ebe2a
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
06cce34  py3: a few more string formatting enhancements in str > repr for floats

comment:10 Changed 10 months ago by
 Status changed from needs_work to needs_review
Don't use Python 3 style fstrings if it will needlessly change the output type of existing functions on Python 2.
comment:11 Changed 10 months ago by
 Reviewers set to Frédéric Chapoton
 Status changed from needs_review to positive_review
ok, let it be. Thanks
comment:12 Changed 10 months ago by
 Status changed from positive_review to needs_work
merge conflict
comment:13 Changed 10 months ago by
 Status changed from needs_work to positive_review
branch is still green, with no apparent conflict with 8.4.b6
I am therefore setting back to positive
comment:14 Changed 10 months ago by
 Branch changed from u/embray/python3/sageplotplot3d/floatrepr to 06cce349f9c8bee1236241fbb53c02202b7ebe2a
 Resolution set to fixed
 Status changed from positive_review to closed
comment:15 Changed 10 months ago by
 Commit 06cce349f9c8bee1236241fbb53c02202b7ebe2a deleted
I wonder where the supposed merge conflict came from then...
comment:16 Changed 10 months ago by
I don't know why Trac does that sometimes...
comment:17 Changed 10 months ago by
Weird. It actually will not even allow me to set the commit field. I guess the plugin doesn't allow a commit field to be set after a ticket is closed, and is so militant about that that it will delete it if you comment on a closed ticket.
Apparently I inserted a tab somewhere??