Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decomposition of some Completely Regular Semigroups into Strong Semilattices of Semigroups #731

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

tomcontileslie
Copy link
Collaborator

@tomcontileslie tomcontileslie commented Jan 13, 2021

May 2021 update: this PR has only ticked one of four boxes, but it works on Clifford semigroups. Implementing IsomorphismSemigroup for other types of completely regular semigroups can be done in a different PR.

This PR introduces methods for decomposing certain completely regular semigroups into strong semilattices, as suggested in Issue #671. The aim is introduce IsomorphismSemigroup methods from the following types of semigroups:

  • Clifford semigroups
  • Normal orthogroups
  • Normal cryptogroups
  • Normal bands (?)

to SSS objects.

@tomcontileslie tomcontileslie force-pushed the sss-decomp branch 2 times, most recently from cfcfe48 to 4301f23 Compare May 20, 2021 10:52
@@ -102,3 +102,6 @@ DeclareProperty("IsSurjectiveSemigroup", IsSemigroup);
InstallTrueMethod(IsSurjectiveSemigroup, IsRegularSemigroup);
InstallTrueMethod(IsSurjectiveSemigroup, IsMonoidAsSemigroup);
InstallTrueMethod(IsSurjectiveSemigroup, IsIdempotentGenerated);

DeclareProperty("IsOrthogroup", IsSemigroup);
DeclareSynonym("IsOrthoGroup", IsOrthogroup);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
DeclareSynonym("IsOrthoGroup", IsOrthogroup);
DeclareSynonymAttr("IsOrthoGroup", IsOrthogroup);

D := DigraphReflexiveTransitiveReduction(Digraph(NaturalPartialOrder(A)));
# currently wrong way round
D := DigraphReverse(D);
N := OutNeighbours(D);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can directly get this via

Suggested change
N := OutNeighbours(D);
N := ReverseNaturalPartialOrder(A);

(although I don't think that necessarily (or at all?) contains x in N[x] for each x - if those are actually necessary, you will still want to add them or modify your code below to acts as if they are there).

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @wilfwilson, unfortunately I get a "no method found" error when running ReverseNaturalPartialOrder on some inputs - for example:

gap> S := Semigroup(Transformation([1, 2, 4, 5, 6, 3, 7, 8]), Transformation([3, 3, 4, 5, 6, 2, 7, 8]), Transformation([1, 2, 5, 3, 6, 8, 4, 4]));
<transformation semigroup of degree 8 with 3 generators>
gap> IsCliffordSemigroup(S);
true
gap> A := Semigroup(Idempotents(S));
<transformation monoid of degree 8 with 3 generators>
gap> NaturalPartialOrder(A);
[ [ 2, 3, 4 ], [ 4 ], [ 4 ], [  ] ]
gap> ReverseNaturalPartialOrder(A);
Error, no method found! For debugging hints type ?Recovery from NoMethodFound
Error, no 2nd choice method found for `ReverseNaturalPartialOrder' on 1 arguments at /Applications/GAP/lib/methsel2.g:250 called from
<function "HANDLE_METHOD_NOT_FOUND">( <arguments> )
 called from read-eval loop at *stdin*:18
type 'quit;' to quit to outer loop

Although you are correct that we don't need x in N[x] since the homomorphisms in this case are the identity, and the strong semilattice constructor is clever enough to fill that in upon creation.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, I'm realising that taking the reflexive transitive reduction of D is redundant, since the SSS constructor effectively reverts this, so I'll remove that part

Comment on lines 369 to 374
for j in N[i] do
map := function(elm)
return idemp * elm;
end;
Add(H[i], MappingByFunction(L[j], L[i], map));
od;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like there is no need to keep re-creating the function map inside the for loop.

Suggested change
for j in N[i] do
map := function(elm)
return idemp * elm;
end;
Add(H[i], MappingByFunction(L[j], L[i], map));
od;
map := elm -> idemp * elm;
for j in N[i] do
Add(H[i], MappingByFunction(L[j], L[i], map));
od;
# Add(H[i], MappingByFunction(L[i], L[i], map)); # Add this if you do actually need `i` to be in `N[i]`

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants