Skip to content

Commit

Permalink
FIX: Align ordering with Magma for orders 3^7,5^7,7^7,11^7
Browse files Browse the repository at this point in the history
The permutations applied had been wrongly implemented and were based on
wrong data. New permutations, based on explicitly comparing with Magma data
were used to determine correct permutations.

The test for the permutations in `ordering.tst` was changed, as the old test
relied on the permutations having small support, which is not true any
longer.

This will fix gap-packages#38

Removed the old tests as they required that the permutations were of small
support. (They also were wrong)

Fixed hashmark and added tests

Added documentation for the new ordering.
  • Loading branch information
hulpke committed Sep 20, 2019
1 parent f4f47c8 commit 5f1d813
Show file tree
Hide file tree
Showing 5 changed files with 545 additions and 76 deletions.
4 changes: 2 additions & 2 deletions PackageInfo.g
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ SetPackageInfo( rec(

PackageName := "SmallGrp",
Subtitle := "The GAP Small Groups Library",
Version := "1.3",
Date := "09/04/2018", # dd/mm/yyyy format
Version := "1.4",
Date := "12/09/2019", # dd/mm/yyyy format
License := "Artistic-2.0",

Persons := [
Expand Down
8 changes: 8 additions & 0 deletions doc/overview.xml
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,14 @@ Groups library replaces the Gap 3 library of solvable groups of order at
most 100. However, both the organisation and data descriptions of these
groups has changed in the Small Groups library.
<P/>

<P>
As of version 1.4 of this library, the arrangement of groups is the same as
in Magma, Version 2.23. In earlier release the ordering in degrees
<M>p^7</M>, <M>p=3,5,7,11</M> disagreed. An attempt to fix this was instated
on version 1.1 of this library, but a wrong permutation was used. If you
would like to refer to index numbers for these orders in older versions
of the library, see <Ref Sect="SMALL_GROUPS_OLD_ORDER"/>).
</Section>

<Section Label="Reference">
Expand Down
Loading

0 comments on commit 5f1d813

Please sign in to comment.