-
Notifications
You must be signed in to change notification settings - Fork 163
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Better define the result of MappingPermListList
This fixes some bugs introduced in moving MappingPermListList into the kernel: * Crash when given negative integers. * Crash when given non-integer types. * Sometimes (incorrectly) returns a permutation when given two lists where no perm exists mapping one to the other. This function also alters the permutation generated by MappingPermListList to only move points in one of the input lists. This fixes a bug in RepresentativeAction, which assumed this was the case.
- Loading branch information
1 parent
8155711
commit 9ad42c9
Showing
4 changed files
with
130 additions
and
22 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
# These functions all worked incorrectly when given symmetric or alternating groups | ||
# Which were not not defined on a domain of the form [1..n] | ||
gap> RepresentativeAction(SymmetricGroup([5,7,11,15]),[7,11],[5,15],OnTuples); | ||
(5,7)(11,15) | ||
gap> RepresentativeAction(AlternatingGroup([5,7,11,15]),[7,11],[5,15],OnTuples); | ||
(5,7)(11,15) | ||
gap> RepresentativeAction(SymmetricGroup([5,7,11,15]),[7,11],[5,15],OnSets); | ||
(5,7)(11,15) | ||
gap> RepresentativeAction(AlternatingGroup([5,7,11,15]),[7,11],[5,15],OnSets); | ||
(5,7)(11,15) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters