unbound the lazy_list formatter

[dkrenn]

In #21164, a string formatter for lazy lists was introduced in the class lazy_list_generic. The aim of this ticket is to unbound this method, so that one is able to use it outside lazy lists as well.

Depends on #21164

Component: misc

Author: Daniel Krenn

Branch/Commit: a8a006a

Reviewer: Clemens Heuberger

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

[dkrenn]

Dependencies: #21164

[dkrenn]

comment:1

This will become handy when using it in conjunction with k-regular sequences (no ticket yet) and probably other sequences as well.

[dkrenn]

Branch: u/dkrenn/lazy_list-unbound-formatter

[dkrenn]

Last 10 new commits:

bc1bd71	Trac #21164, comment 8, 1: make update_cache_up_to private (i.e., prepend a _)
608332f	Trac #21164, comment 8, 2: revert rewrite in get (variable n vs. i)
ef28cce	Trac #21164, comment 11, 3: revert rewrite in __add__
7675c49	Trac #21164, comment 11, 2: refine docstring to make idempotence of lazy_list visible
37c8fad	Trac #21164, comment 11, 4: better description of arguments in str
9ba1dcf	Trac #21164, comment 11, 1: extend docstring explaining inheriting
4ba5bd6	Trac #21200: move code (no other changes)
48d82de	Trac #21200: modify code so that everything works again
e3455ba	Trac #21200: fix doctests
9dc6624	Trac #21200: polish docstrings

[dkrenn]

Commit: 9dc6624

[dkrenn]

Author: Daniel Krenn

[sagetrac-git]

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

a46ba93

Trac #21200: rephrase requirements for input

[sagetrac-git]

Changed commit from 9dc6624 to a46ba93

[dkrenn]

comment:7

Replying to @dkrenn:

This will become handy when using it in conjunction with k-regular sequences (no ticket yet) and probably other sequences as well.

Meanwhile there is a ticket, namely #21202.

[sagetrac-git]

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

d3e2df6

Trac #21200: simplify requirements for input (only iterable needed)

[sagetrac-git]

Changed commit from a46ba93 to d3e2df6

[dkrenn]

comment:9

Simplifed the requirements needed for the input. (Now L needs only to be iterable.)

[videlec]

comment:10

Instead of '{!r}'.format(e) you could use the more direct repr(e).

[videlec]

comment:11

Why don't you encode the default argument in the function

def lazy_list_formatter(L, name='lazy list', separator=', ', more='...',
        opening_delimiter='[', closing_delimiter=']',
        preview=3):

[sagetrac-git]

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

18e1348	Trac #21164: 64bit vs 32bite
90ffe8c	Merge branch 'u/dkrenn/lazy_list-various' into u/dkrenn/lazy_list-unbound-formatter
973e74c	Trac #21200: simplify formatting
f6a1967	Trac #21200: simplify default argument processing

[sagetrac-git]

Changed commit from d3e2df6 to f6a1967

[dkrenn]

comment:13

Replying to @videlec:

Instead of '{!r}'.format(e) you could use the more direct repr(e).

Indeed. Changed.

[dkrenn]

comment:14

Replying to @videlec:

Why don't you encode the default argument in the function

def lazy_list_formatter(L, name='lazy list', separator=', ', more='...',
        opening_delimiter='[', closing_delimiter=']',
        preview=3):

I see that this change makes the code much clearer; thus changed.

[cheuberg]

Changed branch from u/dkrenn/lazy_list-unbound-formatter to u/cheuberg/lazy_list-unbound-formatter

[sagetrac-git]

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

a8a006a

Trac #21200: remove period in INPUT section

[sagetrac-git]

Changed commit from f6a1967 to a8a006a

[cheuberg]

Reviewer: Clemens Heuberger

[cheuberg]

comment:17

@videlec: It seems that you reviewed the code some months ago and that your comments have been taken into account. Feel free to add your name to the Reviewers field.

[vbraun]

Changed branch from u/cheuberg/lazy_list-unbound-formatter to a8a006a