Skip to content

Commit

Permalink
semieunit: fix to String for an MTSE
Browse files Browse the repository at this point in the history
  • Loading branch information
wilfwilson authored and ChristopherRussell committed Aug 30, 2017
1 parent 58dd0a1 commit 44344f7
Show file tree
Hide file tree
Showing 3 changed files with 33 additions and 10 deletions.
2 changes: 1 addition & 1 deletion doc/z-chap12.xml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
For inverse semigroups these two conditions are equivalent. We are only
interested in <E>E-unitary inverse semigroups</E>.
Before defining McAlister triple semigroups we define a McAlister triple.
A <E>McAlister triple</E> is a triple <C>(G,X,Y)</C> which consist of:
A <E>McAlister triple</E> is a triple <C>(G,X,Y)</C> which consists of:

<List>
<Item>
Expand Down
15 changes: 10 additions & 5 deletions gap/semigroups/semieunit.gi
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,14 @@ function(S)
X := McAlisterTripleSemigroupPartialOrder(S);
Y := McAlisterTripleSemigroupSemilattice(S);
return Concatenation("McAlisterTripleSemigroup(", String(G), ", ",
String(X), ", ", String(Y), ")");
String(X), ", ", String(DigraphVertexLabels(Y)), ")");
end);

InstallMethod(PrintObj, "for a McAlister triple semigroup",
[IsMcAlisterTripleSemigroup],
function(S)
Print(String(S));
return;
end);

#TODO Linebreak hints
Expand Down Expand Up @@ -308,10 +315,8 @@ end);
InstallMethod(String, "for a McAlister triple semigroup element rep",
[IsMcAlisterTripleSemigroupElementRep],
function(x)
return Concatenation("MTSE(", String(x![3]), ", ",
String(DigraphVertexLabels(
McAlisterTripleSemigroupSemilattice(x![3]))[x[1]]),
", ", String(x[2]), ")");
return Concatenation("MTSE(", String(x![3]), ", ", String(x![1]), ", ",
String(x![2]), ")");
end);

#TODO Linebreak hints
Expand Down
26 changes: 22 additions & 4 deletions tst/standard/semieunit.tst
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,10 @@ gap> M = M1;
true
gap> String(M);
"McAlisterTripleSemigroup(SymmetricGroup( [ 2 .. 5 ] ), Digraph( [ [ 1 ], [ 1,\
2 ], [ 1, 3 ], [ 1, 4 ], [ 1, 5 ] ] ), Digraph( [ [ 1 ], [ 1, 2 ], [ 1, 3 ], \
[ 1, 4 ] ] ))"
2 ], [ 1, 3 ], [ 1, 4 ], [ 1, 5 ] ] ), [ 1 .. 4 ])"
gap> Print(M, "\n");
McAlisterTripleSemigroup(SymmetricGroup( [ 2 .. 5 ] ), Digraph( [ [ 1 ], [ 1, \
2 ], [ 1, 3 ], [ 1, 4 ], [ 1, 5 ] ] ), [ 1 .. 4 ])

#T# McAlisterTripleSemigroup with bad inputs
gap> G1 := FreeGroup(1);;
Expand Down Expand Up @@ -143,8 +145,7 @@ true
gap> elms := Enumerator(M);;
gap> String(elms[1]);
"MTSE(McAlisterTripleSemigroup(SymmetricGroup( [ 2 .. 5 ] ), Digraph( [ [ 1 ],\
[ 1, 2 ], [ 1, 3 ], [ 1, 4 ], [ 1, 5 ] ] ), Digraph( [ [ 1 ], [ 1, 2 ], [ 1, \
3 ], [ 1, 4 ] ] )), 1, ())"
[ 1, 2 ], [ 1, 3 ], [ 1, 4 ], [ 1, 5 ] ] ), [ 1 .. 4 ]), 1, ())"
gap> OneImmutable(M);
Error, Semigroups: OneImutable (for McAlister triple semigroup): usage,
the argument must be a monoid,
Expand Down Expand Up @@ -236,6 +237,18 @@ gap> M5 := McAlisterTripleSemigroup(G, x1, x1, act);;
gap> IsomorphismSemigroups(M1, M5);
fail

#T# IsomorphismSemigroups, where RepresentativeAction fails
gap> gr := DigraphFromDigraph6String("+H_A?GC_Q@G~wA?G");
<digraph with 9 vertices, 20 edges>
gap> G := Group((1, 2, 3)(4, 5, 6), (8, 9));
Group([ (1,2,3)(4,5,6), (8,9) ])
gap> S1 := McAlisterTripleSemigroup(G, gr, [1, 4, 5, 7, 8]);
<McAlister triple semigroup over Group([ (1,2,3)(4,5,6), (8,9) ])>
gap> S2 := McAlisterTripleSemigroup(G, gr, [3, 6, 7, 8, 9]);
<McAlister triple semigroup over Group([ (1,2,3)(4,5,6), (8,9) ])>
gap> IsomorphismSemigroups(S1, S2);
fail

#T# IsIsomorphicSemigroup
gap> IsIsomorphicSemigroup(M1, M1);
true
Expand Down Expand Up @@ -271,6 +284,11 @@ gap> IsFInverseMonoid(M);
true
gap> IsFInverseSemigroup(M);
true
gap> S := McAlisterTripleSemigroup(Group((4, 5)),
> Digraph([[1], [1, 2], [1, 3], [1, 2, 3, 4], [1, 2, 3, 5]]), [1 .. 4]);
<McAlister triple semigroup over Group([ (4,5) ])>
gap> IsFInverseSemigroup(S);
false

#T# EUnitaryInverseCover
gap> S := InverseMonoid([PartialPermNC([1, 3], [1, 3]),
Expand Down

0 comments on commit 44344f7

Please sign in to comment.