Poset documentation polishing: Graphs

[jm58660]

Check documentation for poset functions that returns a (di)graph.

Changes are minor things, just unifying some wording, moving examples that are clearly tests to tests-block etc. Also

• Remove wrapped-parameter from hasse_diagram(), as it did nothing.
• Fix typo in lattices.py, "quantum" was in wrong place.
• Move breath() last in list in lattices.py, as other functions return a Boolean value.

This continues the serie of #18925, #18941, #18959, #19141, #19360, #19435.

CC: @kevindilks @darijgr

Component: documentation

Author: Jori Mäntysalo

Branch/Commit: cd39820

Reviewer: Frédéric Chapoton

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

[jm58660]

Branch: u/jmantysalo/poset-graphs

[sagetrac-git]

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

de0e9db

Some bikeshedding to poset functions that return a graph.

[sagetrac-git]

Commit: de0e9db

[jm58660]

comment:3

Kevin, I am continuing this serie. Compiling now, so not ready for review yet.

Darij: An example in frank_network contains numbers 12, -13, 14 and 16. Is there some reason for that?

[sagetrac-git]

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

cd39820

A correction.

[sagetrac-git]

Changed commit from de0e9db to cd39820

[jm58660]

comment:5

Kevin, maybe you can check this; we can check come back to frank_network later.

...polynomials next... this serie seems to take a year or two.

[jm58660]

comment:6

Just pinging.

[fchapoton]

Reviewer: Frédéric Chapoton

[fchapoton]

comment:7

ok, let it be. Even if I do not see the point of changing all the doctests..

[kevindilks]

comment:8

Are you able to run the doctests? After applying this ticket, I was able to run doctests for unchanged files and lattices.py, but when I tried to test posets.py it just hangs until I force quit, where it shows it was using 0 CPU time. Currently upgrading from 7.4beta1 to 7.3beta3 to see if that makes a difference.

[fchapoton]

comment:9

hum, one patchbot is happy with the doctests, so I did not double check.

There was a problem indeed about some test in posets.py in 7.4.beta(1-2), that has been solved in 7.4.b3

[kevindilks]

comment:10

Alright, I just finished updating to 7.4b3 and it does work now.

[jm58660]

comment:11

Thanks!

I am gradually checking posets.py, and trying to make meaningful examples in the process. Next will be polynomials; for example order_polynomial() should use as_ideals=False when calling order_ideals_lattice() and so on.

[vbraun]

Changed branch from u/jmantysalo/poset-graphs to cd39820