Opened 12 years ago
Closed 12 years ago
#9810 closed defect (fixed)
Heisenbug in RationalPolyhedralCone.facets
| Reported by: | vbraun | Owned by: | mhampton |
|---|---|---|---|
| Priority: | major | Milestone: | sage-4.6 |
| Component: | geometry | Keywords: | |
| Cc: | novoselt | Merged in: | sage-4.6.alpha1 |
| Authors: | Andrey Novoseltsev | Reviewers: | Volker Braun |
| Report Upstream: | N/A | Work issues: | |
| Branch: | Commit: | ||
| Dependencies: | Stopgaps: |
Status badges
Description
The faces method of a cone in a fan manages to screw up subsequent facet output:
sage: cone = toric_varieties.dP8().fan().generating_cone(0) sage: cone 2-d cone of Rational polyhedral fan in 2-d lattice N sage: cone.facets() (1-d cone of Rational polyhedral fan in 2-d lattice N, 1-d cone of Rational polyhedral fan in 2-d lattice N) sage: cone.faces() ((0-d cone of Rational polyhedral fan in 2-d lattice N,), (1-d cone of Rational polyhedral fan in 2-d lattice N, 1-d cone of Rational polyhedral fan in 2-d lattice N), (2-d cone of Rational polyhedral fan in 2-d lattice N,)) sage: cone.facets() (2-d cone of Rational polyhedral fan in 2-d lattice N,)
This is on vanilla Sage 4.5.2 without any patches applied. Andrey, I think its somewhere in your code so you'll probably find it faster than I can.
Attachments (1)
Change History (5)
Changed 12 years ago by
| Attachment: | trac_9810_cone_faces_invalidates_facets.patch added |
|---|
comment:1 Changed 12 years ago by
| Authors: | → Andrey Novoseltsev |
|---|---|
| Status: | new → needs_review |
comment:2 Changed 12 years ago by
| Reviewers: | → Volker Braun |
|---|---|
| Status: | needs_review → positive_review |
Fixes it!
comment:3 Changed 12 years ago by
| Milestone: | → sage-4.5.3 |
|---|
comment:4 Changed 12 years ago by
| Merged in: | → sage-4.6.alpha1 |
|---|---|
| Resolution: | → fixed |
| Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.
Should be 2 instead of 1... Thanks for catching!