Opened 11 months ago
Closed 9 months ago
#30545 closed defect (fixed)
A 1dimensional polytope has no edge
Reported by:  vpilaud  Owned by:  

Priority:  minor  Milestone:  sage9.3 
Component:  geometry  Keywords:  polytopes 
Cc:  jipilab, chapoton, ghkliem  Merged in:  
Authors:  Frédéric Chapoton  Reviewers:  Samuel Lelièvre 
Report Upstream:  N/A  Work issues:  
Branch:  4c3016a (Commits, GitHub, GitLab)  Commit:  4c3016a60af07b5bbb5cf0b1f3e161cfdb648895 
Dependencies:  Stopgaps: 
Description (last modified by )
The graph of a polytope used to be constructed
in the method graph
using pairs of vertices
satisfying common inequalities of the inequalities
list, which did not work in dimension 1:
In Sage 9.0:
sage: polytopes.simplex(1).graph().edges() []
This has since been fixed.
This ticket adds a doctest.
Change History (13)
comment:1 Changed 11 months ago by
 Cc jipilab chapoton ghkliem added; labbe@… chapoton@… removed
 Milestone changed from sage9.2 to sage9.3
comment:2 Changed 11 months ago by
comment:3 Changed 11 months ago by
I cannot reproduce this either.
comment:4 Changed 11 months ago by
by the way, I see bouncing emails from trac to vinc.pilaud@... (they seem to be notifications on this ticket).
Please alert vpilaud.
Yours, postmaster :)
comment:5 Changed 11 months ago by
I had that on version 9.0 and was too lazy to update. So nevermind. I took the opportunity to update my email address on track.
comment:6 Changed 11 months ago by
 Milestone changed from sage9.3 to sageduplicate/invalid/wontfix
 Status changed from new to needs_info
Two ways to solve this ticket:
 add a doctest for that (is there already one?)
 or set to duplicate/invalid/wontfix
comment:7 Changed 11 months ago by
Cheap ways to check whether something is fixed in a more recent Sage version than you have:
 the SageCell server usually runs the last stable version of Sage (currently Sage 9.1)
 The main CoCalc instance has a lot of Sage versions installed, including the last few stable versions, and usually a recent enough development version.
This can help decide whether to upgrade.
comment:8 Changed 11 months ago by
 Branch set to u/chapoton/30545
 Commit set to 4c3016a60af07b5bbb5cf0b1f3e161cfdb648895
 Status changed from needs_info to needs_review
Here is a doctest
New commits:
4c3016a  add a check for the vertex graph af the linesegment polytope

comment:9 Changed 11 months ago by
 Milestone changed from sageduplicate/invalid/wontfix to sage9.2
 Status changed from needs_review to positive_review
Thanks!
comment:10 Changed 11 months ago by
 Description modified (diff)
 Reviewers set to Samuel Lelièvre
comment:11 Changed 11 months ago by
Thanks Samuel and Frédéric for the links and resolution of the ticket.
comment:12 Changed 9 months ago by
 Milestone changed from sage9.2 to sage9.3
comment:13 Changed 9 months ago by
 Branch changed from u/chapoton/30545 to 4c3016a60af07b5bbb5cf0b1f3e161cfdb648895
 Resolution set to fixed
 Status changed from positive_review to closed
Vincent, tu utilises quelle version de sage ? Ca marche pour moi dans la version 9.2.beta12.