Opened 6 years ago
Closed 6 years ago
#21197 closed enhancement (fixed)
Poset documentation polishing: Graphs
Reported by:  Jori Mäntysalo  Owned by:  

Priority:  minor  Milestone:  sage7.4 
Component:  documentation  Keywords:  
Cc:  Kevin Dilks, darij  Merged in:  
Authors:  Jori Mäntysalo  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  cd39820 (Commits, GitHub, GitLab)  Commit:  cd3982022709c7fcb322a38ebadd7f4babe5b256 
Dependencies:  Stopgaps: 
Description (last modified by )
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 testsblock etc. Also
 Remove
wrapped
parameter fromhasse_diagram()
, as it did nothing.  Fix typo in
lattices.py
, "quantum" was in wrong place.  Move
breath()
last in list inlattices.py
, as other functions return a Boolean value.
This continues the serie of #18925, #18941, #18959, #19141, #19360, #19435.
Change History (12)
comment:1 Changed 6 years ago by
Branch:  → u/jmantysalo/posetgraphs 

comment:2 Changed 6 years ago by
Commit:  → de0e9dbb3b8c66cd871caa96c2824943399ddc88 

comment:3 Changed 6 years ago by
Cc:  Kevin Dilks darij added 

Description:  modified (diff) 
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?
comment:4 Changed 6 years ago by
Commit:  de0e9dbb3b8c66cd871caa96c2824943399ddc88 → cd3982022709c7fcb322a38ebadd7f4babe5b256 

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

comment:5 Changed 6 years ago by
Status:  new → needs_review 

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.
comment:7 Changed 6 years ago by
Reviewers:  → Frédéric Chapoton 

Status:  needs_review → positive_review 
ok, let it be. Even if I do not see the point of changing all the doctests..
comment:8 Changed 6 years ago by
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.
comment:9 Changed 6 years ago by
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(12), that has been solved in 7.4.b3
comment:11 Changed 6 years ago by
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.
comment:12 Changed 6 years ago by
Branch:  u/jmantysalo/posetgraphs → cd3982022709c7fcb322a38ebadd7f4babe5b256 

Resolution:  → fixed 
Status:  positive_review → closed 
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
Some bikeshedding to poset functions that return a graph.