{"action":"created","issue":{"url":"https://api.github.com/repos/gap-system/gap/issues/2276","repository_url":"https://api.github.com/repos/gap-system/gap","labels_url":"https://api.github.com/repos/gap-system/gap/issues/2276/labels{/name}","comments_url":"https://api.github.com/repos/gap-system/gap/issues/2276/comments","events_url":"https://api.github.com/repos/gap-system/gap/issues/2276/events","html_url":"https://github.com/gap-system/gap/pull/2276","id":307241923,"number":2276,"title":"Not all finitely generated magmas are groups","user":{"login":"fingolfin","id":241512,"avatar_url":"https://avatars2.githubusercontent.com/u/241512?v=4","gravatar_id":"","url":"https://api.github.com/users/fingolfin","html_url":"https://github.com/fingolfin","followers_url":"https://api.github.com/users/fingolfin/followers","following_url":"https://api.github.com/users/fingolfin/following{/other_user}","gists_url":"https://api.github.com/users/fingolfin/gists{/gist_id}","starred_url":"https://api.github.com/users/fingolfin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fingolfin/subscriptions","organizations_url":"https://api.github.com/users/fingolfin/orgs","repos_url":"https://api.github.com/users/fingolfin/repos","events_url":"https://api.github.com/users/fingolfin/events{/privacy}","received_events_url":"https://api.github.com/users/fingolfin/received_events","type":"User","site_admin":false},"labels":[{"id":180442294,"url":"https://api.github.com/repos/gap-system/gap/labels/bug","name":"bug","color":"e11d21","default":true},{"id":300484804,"url":"https://api.github.com/repos/gap-system/gap/labels/bug:%20wrong%20result","name":"bug: wrong result","color":"e11d21","default":false},{"id":577473073,"url":"https://api.github.com/repos/gap-system/gap/labels/library","name":"library","color":"d4c5f9","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-03-21T13:22:21Z","updated_at":"2018-03-21T14:30:51Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/gap-system/gap/pulls/2276","html_url":"https://github.com/gap-system/gap/pull/2276","diff_url":"https://github.com/gap-system/gap/pull/2276.diff","patch_url":"https://github.com/gap-system/gap/pull/2276.patch"},"body":"... hence we should not set IsFinitelyGeneratedGroup for them. Instead, only set IsFinitelyGeneratedGroup if the family indicates that its elements are associative.\r\n\r\nNote that this all is only done to avoid a call to the default immediate method for IsFinitelyGeneratedGroup. I am not really sure whether this is worth the hassle... Perhaps it is when one creates thousands and thousands of groups in a tight loop?\r\n\r\nFixes #2252 "},"comment":{"url":"https://api.github.com/repos/gap-system/gap/issues/comments/374957231","html_url":"https://github.com/gap-system/gap/pull/2276#issuecomment-374957231","issue_url":"https://api.github.com/repos/gap-system/gap/issues/2276","id":374957231,"user":{"login":"fingolfin","id":241512,"avatar_url":"https://avatars2.githubusercontent.com/u/241512?v=4","gravatar_id":"","url":"https://api.github.com/users/fingolfin","html_url":"https://github.com/fingolfin","followers_url":"https://api.github.com/users/fingolfin/followers","following_url":"https://api.github.com/users/fingolfin/following{/other_user}","gists_url":"https://api.github.com/users/fingolfin/gists{/gist_id}","starred_url":"https://api.github.com/users/fingolfin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fingolfin/subscriptions","organizations_url":"https://api.github.com/users/fingolfin/orgs","repos_url":"https://api.github.com/users/fingolfin/repos","events_url":"https://api.github.com/users/fingolfin/events{/privacy}","received_events_url":"https://api.github.com/users/fingolfin/received_events","type":"User","site_admin":false},"created_at":"2018-03-21T14:30:51Z","updated_at":"2018-03-21T14:30:51Z","author_association":"OWNER","body":"Anyway, I'll prepare an alternative PR introducing `IsFinitelyGeneratedMagma`."}}
{"actor":{"display_login":"fingolfin"}}