#18454: New `random_cone()` function
-------------------------------------+-------------------------------------
Reporter: mjo | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.8
Component: geometry | Resolution:
Keywords: | Merged in:
Authors: Michael Orlitzky | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/mjo/ticket/18454 | 7bdaf1cd51d8f6f3007f11888991cefc186a4ee1
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by novoselt):
I have a problem with `min/max_dim` - I was sure they talk about the
dimension of the cone, until I got to their description. The example with
it is also confusing:
{{{
We can predict the dimension when ``min_dim == max_dim``::
sage: set_random_seed()
sage: random_cone(min_dim=4, max_dim=4, min_rays=0, max_rays=0)
0-d cone in 4-d lattice N
}}}
To me this looks like we predict the dimension 0 based on `min/max_rays`.
I would be in favour of using `min/max_dim` for the dimension of the cone,
allowing specifying `lattice` together with these parameters as long as
its dimension is at least `max_dim`, and when not given defaulting to
`max_dim`-dimensional lattice. If you don't want to work on such
functionality, can we please rename parameters to `min/max_ambient_dim` to
avoid ambiguity when reading calls of this function?
--
Ticket URL: <http://trac.sagemath.org/ticket/18454#comment:19>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
