#4341: [with patch, positive review] Optimisations + corrections to latin.py
-----------------------------+----------------------------------------------
Reporter: carlohamalainen | Owner: mhansen
Type: enhancement | Status: new
Priority: minor | Milestone: sage-3.2.1
Component: combinatorics | Resolution:
Keywords: |
-----------------------------+----------------------------------------------
Old description:
> * Removed code that used gap.Representative in an unsafe manner (assumed
> that the ordering would be the same on each execution but the GAP manual
> says that this may not be the case). Previous code did work but was not
> safe.
>
> * Replacement tau_to_bitrade uses correct and straightforward
> combinatorial approach.
>
> * Replacement of p3_group_bitrade_generators is orders of magnitude
> faster; uses GAP's IsomorphismPermGroup instead of explicitly
> constructing a natural homomorphism.
New description:
* Removed code that used gap.Representative in an unsafe manner (assumed
that the ordering would be the same on each execution but the GAP manual
says that this may not be the case). Previous code did work but was not
safe.
* Replacement tau_to_bitrade uses correct and straightforward
combinatorial approach.
* Replacement of p3_group_bitrade_generators is orders of magnitude
faster; uses GAP's IsomorphismPermGroup instead of explicitly constructing
a natural homomorphism.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4341#comment:4>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
