Changes between Version 4 and Version 7 of Ticket #7729
- Timestamp:
- Dec 19, 2009, 12:23:57 AM (13 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #7729
-
Property
Status
changed from
new
toneeds_work
-
Property
Component
changed from
algebra
tocombinatorics
-
Property
Status
changed from
-
Ticket #7729 – Description
v4 v7 9 9 }}} 10 10 11 This code is very tested for type A and is almost certainly correct for Weyl groups of finite type. I have not tried it for any affine Weyl groups.11 This code is very tested for type A and is almost certainly correct for Weyl groups of finite type. It also works for affine Weyl groups in the most recently posted version. 12 12 13 13 The following issues remain. 14 14 15 15 * David Roe suggested that the _coerce_impl method should be removed. I have not looked at this yet. 16 17 * It should be made to work with Affine Weyl groups. I have not checked whether this requires further modification.18 16 19 17 * Subjectively, it seems a little slow compared with a previous implementation for type A only. This is probably a limitation of the {{{WeylGroup()}}} class on which it depends. My earlier implementation was based on Permutation. If it proves unacceptably slow it may be possible to speed it up by a caching scheme. … … 23 21 For some further discussion of this topic see 24 22 http://groups.google.com/group/sage-combinat-devel/browse_thread/thread/78fc23f23cafe705?hl=en 25 26 It is well tested for type A and is probably correct for all Cartan Types of finite type.