#9972: Add toric lattice morphisms
-------------------------------------+--------------------------------------
Reporter: novoselt | Owner: mhampton
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-4.6
Component: geometry | Keywords:
Author: Andrey Novoseltsev | Upstream: N/A
Reviewer: Volker Braun | Merged:
Work_issues: switch to FanMorphism |
-------------------------------------+--------------------------------------
Comment(by novoselt):
Well, if you just want to add `fan.ray_indices(cone)` method (which can be
potentially slow), I am OK with it, although I'd rather not ;-) But I
really want to keep `cone.ambient_ray_indices()` as is without any
arguments, especially forced ones.
And I still think that the best way is to explicitly convert input cones
to cones of a particular fan and then freely use any fan-dependent
methods. Given the current size of toric code it does not seem to me that
we have any particularly dangerous methods (the most dangerous are
`check=False` and `normalize=False` options, which are described as
dangerous). So maybe instead of changing code we can put a warning in the
documentation that this method may be dangerous and one should ensure that
`cone.ambient()` is what it should be before calling this method. It seems
to me that it is quite difficult to start using `ambient_ray_indices`
without reading its documentation at least once.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9972#comment:25>
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.
