Enumerator needs to be fixed

[vendramin]

YangBaxter/gap/skew.gi

Line 134 in 43b6a6f

InstallMethod(Enumerator,

[olexandr-konovalov]

For now, we are just returning the list of elements, what's not suitable in the long run.

[olexandr-konovalov]

The way of doing this would be to use Enumerator of the underlying group and then use the embedding of its element into the brace. I am not sure if we need this though - we do store all lists when creating the brace anyway? Perhaps we can simply delete Enumerator instead.

[vendramin]

We just need to be able to do things like for x in brace do… What do you think would be the best way to go?

…

On 18 May 2023, at 14:42, Olexandr Konovalov ***@***.***> wrote: The way of doing this would be to use Enumerator of the underlying group and then use the embedding of its element into the brace. I am not sure if we need this though - we do store all lists when creating the brace anyway? Perhaps we can simply delete Enumerator instead. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: ***@***.***>

[olexandr-konovalov]

Ah, you're right! Perhaps current code is ok then.