some details in shuffle and multizeta

[fchapoton]

trying to make the shuffle simpler and the product of MZV slightly faster

Component: combinatorics

Author: Frédéric Chapoton

Branch/Commit: 7229a19

Reviewer: David Coudert

Issue created by migration from https://trac.sagemath.org/ticket/33102

[fchapoton]

Branch: u/chapoton/33102

[fchapoton]

Commit: 7229a19

[fchapoton]

New commits:

7229a19

some details about shuffle of words and multizetas

[dcoudert]

comment:3

a minor detail, isn't it better to use itertools.chain here

-            w1_parent(list(self._w1) + list(self._w2), check=self._check)
+            w1_parent(list(itertools.chain(self._w1, self._w2)), check=self._check)

[fchapoton]

comment:4

Salut, bonne année. Merci pour le commentaire. Je ne suis pas très convaincu qu'on gagne quelque chose à utiliser itertools ici. C'est en dehors de la boucle principale.

[dcoudert]

comment:5

Bonne année. Effectivement, le gain n'est pas clair du tout donc autant rester simple.

LGTM.

[dcoudert]

Reviewer: David Coudert

[slel]

comment:6

Bonne année. En option on pourrait changer au passage:

-            sage: S.list() #indirect test
+            sage: S.list()  # indirect doctest

[vbraun]

Changed branch from u/chapoton/33102 to 7229a19