#9296: Add lattice computations for convex polyhedral cones
----------------------------------+-----------------------------------------
Reporter: vbraun | Owner: mhampton
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-4.5
Component: geometry | Keywords:
Author: Volker Braun | Upstream: N/A
Reviewer: Andrey Novoseltsev | Merged:
Work_issues: |
----------------------------------+-----------------------------------------
Comment(by vbraun):
2) I think `is_trivial` then belongs to `ConvexRationalPolyhedralCone`
(where I put it originally). The meaning of "trivial fan" usually depends
on the context.
3) I see your point, but
* Since there is a `N_basis` and `N_quotient` it should be clear that
the first is `N_\sigma` and the second is `N(\sigma)` in the standard
notation. And yes, the standard notation has its quirks, but I think we
should adhere to it.
* once you start with `sigma.spanned_latttice()` you should write
`sigma.spanned_lattice_perpendicular_basis()` for `M(\sigma)`. Too long
for my taste.
* none of the lattice computations are particularly useful for
beginners, they will usually be used by other methods to compute geometric
quantities.
* multiple interfaces for the same quantity would be against the python
spirit.
4,5) Makes sense, I added the `_basis` suffix where appropriate.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9296#comment:6>
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.
