#13590: Fix random simplicial complexes
--------------------------------------+-------------------------------------
Reporter: jhpalmieri | Owner: John Palmieri
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-5.5
Component: algebraic topology | Resolution:
Keywords: simplicial complex | Work issues:
Report Upstream: N/A | Reviewers:
Authors: John Palmieri | Merged in:
Dependencies: | Stopgaps:
--------------------------------------+-------------------------------------
Description changed by jhpalmieri:
Old description:
> Random simplicial complexes don't have the right number of vertices. From
> the doctests:
>
> sage: simplicial_complexes.RandomComplex(6, 2)
> Simplicial complex with vertex set (0, 1, 2, 3, 4, 5, 6) and
> 15 facets
> sage: simplicial_complexes.RandomComplex(6,12)
> Simplicial complex with vertex set (0, 1, 2, 3, 4, 5, 6, 7)
> and facets {(0, 1, 2, 3, 4, 5, 6, 7)}
>
> In both cases, these should have 6 vertices, but they have 7 and 8
> instead. Thus this method gives mathematically incorrect answers. The
> attached patch fixes the problems.
New description:
Random simplicial complexes don't have the right number of vertices. From
the doctests:
{{{
sage: simplicial_complexes.RandomComplex(6, 2)
Simplicial complex with vertex set (0, 1, 2, 3, 4, 5, 6) and
15 facets
sage: simplicial_complexes.RandomComplex(6,12)
Simplicial complex with vertex set (0, 1, 2, 3, 4, 5, 6, 7)
and facets {(0, 1, 2, 3, 4, 5, 6, 7)}
}}}
In both cases, these should have 6 vertices, but they have 7 and 8
instead. Thus this method gives mathematically incorrect answers. The
attached patch fixes the problems.
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13590#comment:2>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
