py3: caring for structure/ doctests

[fchapoton]

all but one

Component: python3

Author: Frédéric Chapoton

Branch/Commit: f5394d4

Reviewer: Travis Scrimshaw

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

[fchapoton]

New commits:

82f94e1

py3: some care for structure/ doctests

[fchapoton]

Branch: u/chapoton/26768

[fchapoton]

Commit: 82f94e1

[tscrim]

Reviewer: Travis Scrimshaw

[tscrim]

comment:2

I don't understand why you now get the deprecation warning for that sum. However, since that function is deprecated, I would just revert that change. Once done, you can set a positive review.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

f5394d4

trac 26768 fixing the doctest

[sagetrac-git]

Changed commit from 82f94e1 to f5394d4

[fchapoton]

comment:4

I have instead added the deprecation warning in the doctest.

With this change, all doctests in structure/ pass both with py2 and py3.

So, even if I do no like at all this mysterious appearance of deprecations, I would rather bear with it right now, in order to make good progress towards python3.

[tscrim]

comment:5

That particular change was a micro-optimization/presentation change (unless I am misunderstanding something), so that is why I suggested a revert. However, since this does work, positive review.

[vbraun]

Changed branch from u/chapoton/26768 to f5394d4