pickle the paths of Rauzy diagrams

[videlec]

There is a pickle error with the nested class RauzyDiagram.Path in sage.combinat.iet.template

sage: p = iet.Permutation('a b c','c b a')
sage: r = p.rauzy_diagram()
sage: g = r.path(p, 't', 'b')
sage: dumps(g)
PicklingError Traceback(most recent call last)
...
PicklingError: Can't pickle <class 'sage.combinat.iet.labelled.Path'>: attribute lookup sage.combinat.iet.labelled.Path failed

A __metaclass__ must be defined for RauzyDiagram.

The ticket depends on #8386 which moves the module for interval exchange transformations from sage.combinat.iet to sage.dynamics.interval_exchanges

Apply only attachment: trac_8388_pickling_path.v2.patch

Depends on #8386

CC: @sagetrac-sage-combinat @sagetrac-tmonteil

Component: combinatorics

Keywords: pickle, nested class

Author: Vincent Delecroix

Reviewer: Frédéric Chapoton, Nathann Cohen

Merged: sage-5.13.beta0

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

[videlec]

Changed keywords from pickle of a nested class to pickle, nested class

[hivert]

comment:2

Replying to @videlec:
Do you want it to be reviewed (you didn't check need review). I'm volunteering to review it as soon as #8386 is reviewed.

[fchapoton]

Dependencies: #8386

[fchapoton]

comment:5

please add a commit message to your patch (using sage -hg qrefresh -e) starting with #8388 (to make the bot more happy)

[fchapoton]

comment:6

Does it really depends on #8386 ? The buildbot was happier before I added this dependency.

[fchapoton]

Changed dependencies from #8386 to none

[nathanncohen]

comment:8

Hellooo !!!!

This ticket can be set to positive review after two changes :

• That it be rebased against a recent version of Sage. The patch applies with a hunk right now
• That the commit message be changed so as to contain the ticket's number, or the release manager will have to shout :-D

Nathann