py3: fix the last doctest in geometry

[fchapoton]

or rather tag it with # py2

I have no idea why this does fail only in python3. But anyway, we cannot really do anything with the snub dodecahedron currently.

CC: @jplab @videlec @tscrim

Component: python3

Author: Frédéric Chapoton, John Palmieri

Branch/Commit: 6fcdf05

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/27965

[fchapoton]

Branch: u/chapoton/27965

[fchapoton]

Commit: 7c9a636

[fchapoton]

comment:1

very trivial ticket, please review swiftly

---

New commits:

7c9a636

py3: fix the last doctest in geometry

[jhpalmieri]

comment:2

I think the issue here is not that polytopes.snub_dodecahedron() is broken in Python 3, but rather that Python 3 prints warning messages multiple times, whereas Python 2 only prints them once, at least while doctesting. A change like that in #27612 might be another option:

diff --git a/src/sage/geometry/polyhedron/library.py b/src/sage/geometry/polyhedron/library.py
index 71914d635a..c9bb1f35e8 100644
--- a/src/sage/geometry/polyhedron/library.py
+++ b/src/sage/geometry/polyhedron/library.py
@@ -1489,7 +1489,11 @@ class Polytopes():
         Unfortunately, no polyhedra backend supports the construction of the
         snub dodecahedron at the moment::
 
-            sage: sd = polytopes.snub_dodecahedron()
+            sage: sd = polytopes.snub_dodecahedron() # py2
+            sage: sd = polytopes.snub_dodecahedron() # py3
+            doctest:warning
+            ...
+            UserWarning: This polyhedron data is numerically complicated; cdd could not convert between the inexact V and H representation without loss of data. The resulting object might show inconsistencies.
             sage: sd.f_vector() # not tested
             (1, 60, 150, 92, 1)
             sage: sd.base_ring() # not tested

Given the comment, though ("no polyhedra backend supports the construction of the snub dodecahedron at the moment"), it's not a big deal.

[jhpalmieri]

Changed branch from u/chapoton/27965 to u/jhpalmieri/27965

[jhpalmieri]

Changed author from Frédéric Chapoton to Frédéric Chapoton, John Palmieri

[jhpalmieri]

comment:4

I prefer my suggestion. When we move away from Python 2, it leaves a valid py3 command to execute.

---

New commits:

6fcdf05

trac 27965: py3 fix in geometry/polyhedron/library.py

[jhpalmieri]

Changed commit from 7c9a636 to 6fcdf05

[fchapoton]

Reviewer: Frédéric Chapoton

[fchapoton]

comment:5

ok, good for me.