fix doctest in quantum_group_gap

[fchapoton]

• partial pep8 cleanup of this file

Component: combinatorics

Author: Frédéric Chapoton

Branch/Commit: 130eb19

Reviewer: Michael Orlitzky

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

[fchapoton]

New commits:

130eb19

pep cleanup and doctest fix in quantum_group_gap

[fchapoton]

Branch: u/chapoton/33148

[fchapoton]

Commit: 130eb19

[orlitzky]

comment:2

Isn't reversed(list(enumerate(... equivalent to reversed(enumerate(..., but slower?

[sagetrac-git]

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

a4716f0

simplify use of reversed

[sagetrac-git]

Changed commit from 130eb19 to a4716f0

[fchapoton]

comment:4

true, thanks. Fixed now

[orlitzky]

comment:5

Hi, sorry, I'm an idiot:

TypeError: 'enumerate' object is not reversible

You have to do sorted(enumerate(x), reverse=True), but I've done some testing and that's actually slower regardless of the size of your collection. So you can just drop that last commit. The rest looks good.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

[sagetrac-git]

Changed commit from a4716f0 to 130eb19

[fchapoton]

comment:7

sorry, I should have run the doctests.. I dropped the last commit.

[orlitzky]

comment:8

Replying to @fchapoton:

sorry, I should have run the doctests..

I did, not realizing they were all behind # optional tags...

[orlitzky]

Reviewer: Michael Orlitzky

[slel]

comment:10

Setting milestone to 9.6 now that 9.5 is out.

[vbraun]

Changed branch from u/chapoton/33148 to 130eb19