Opened 3 years ago
Closed 3 years ago
#19023 closed defect (fixed)
Minor typos in 6.8
Reported by:  jmantysalo  Owned by:  

Priority:  trivial  Milestone:  sage6.9 
Component:  documentation  Keywords:  
Cc:  Merged in:  
Authors:  Nathann Cohen, Frédéric Chapoton, Jori Mäntysalo  Reviewers:  Jori Mäntysalo, Nathann Cohen 
Report Upstream:  N/A  Work issues:  
Branch:  712ebe8 (Commits)  Commit:  712ebe89cda1b0e0804a680aac62db60acf9ef7b 
Dependencies:  Stopgaps: 
Description (last modified by )
This is a container ticket for minor documentation typos found in version 6.8. I'll add them here, if I am not doing another ticket for same part of Sage. Of course others can also make additions to this.
 generic_graph.py / breadth_first_search / :meth:
successors
broken link. Same with depth first.  generic_graph.py / is_planar, broken link to sage.graphs.planarity
 To posets/init.html add links also to finite categories of posets and lattices.
 posets.py, relabel(), wrong indentation after INPUT section.
 Group theory tutorial, list of small groups, two
}
Change History (28)
comment:1 Changed 3 years ago by
 Description modified (diff)
comment:2 Changed 3 years ago by
 Description modified (diff)
comment:3 Changed 3 years ago by
 Branch set to u/ncohen/19023
 Commit set to 8ce7f026975f1de95042018eca907553c55964a9
 Status changed from new to needs_review
comment:4 Changed 3 years ago by
 Status changed from needs_review to needs_work
comment:5 Changed 3 years ago by
 Description modified (diff)
comment:6 Changed 3 years ago by
 Description modified (diff)
comment:7 followup: ↓ 8 Changed 3 years ago by
 Branch changed from u/ncohen/19023 to public/ticket/19023
 Commit changed from 8ce7f026975f1de95042018eca907553c55964a9 to f6a4371c5134cca8dfcb324551d95b48f121a45d
comment:8 in reply to: ↑ 7 Changed 3 years ago by
Replying to chapoton:
I took care of the typos in the group theory tuto and posets. Maybe we can just stop there for the moment ?
But then we should just open a new one... And I guess 6.9 will not be out for some time.
I did this because a) Nathann hoped for less tickets, at least tickets only correcting a typo or two, b) Sage seems to still have small typos that I found when reading the manuals, and c) for me it seems unlogical to correct totally nonrelating typo in some ticket.
As an artifical example: Let's suppose that I wonder about Frattini sublattice. When thinking about docstring I see a typo in Frattini subgroup. Now I should make a note about the typo, so that I won't forget it when I make a patch at last.
comment:9 Changed 3 years ago by
 Commit changed from f6a4371c5134cca8dfcb324551d95b48f121a45d to f43f9c4da4378b56cc0bc5e8cdc234754ceaebc1
Branch pushed to git repo; I updated commit sha1. New commits:
f43f9c4  Added links to categories of *finite* posets and lattices.

comment:10 followups: ↓ 11 ↓ 12 Changed 3 years ago by
 Reviewers set to Jori Mäntysalo
 Status changed from needs_work to needs_review
I look throught your pathces. Two questions:
 Why is
interior_paths
removed from index?  What is meant by "(see
GenericGraph
andGenericGraph
)" on documentation of optionforce_spring_layout
?
Otherwise all seems good. As beta5 is already out, maybe we should close this and start collecting typos for 6.10. Please also review my small patch that add two links.
comment:11 in reply to: ↑ 10 ; followup: ↓ 20 Changed 3 years ago by
Yo,
 Why is
interior_paths
removed from index?
Because the function does not exist anymore (#17462)
 What is meant by "(see
GenericGraph
andGenericGraph
)" on documentation of optionforce_spring_layout
?
Odd. Well, I guess it was meant to be references toward .show
and .plot
.
Nathann
comment:12 in reply to: ↑ 10 ; followup: ↓ 13 Changed 3 years ago by
Please also review my small patch that add two links.
To me FinitePosets
and FiniteLatticePosets
are internal classes, and not meant to be instanciated by the user.
Nathann
comment:13 in reply to: ↑ 12 ; followup: ↓ 14 Changed 3 years ago by
Replying to ncohen:
To me
FinitePosets
andFiniteLatticePosets
are internal classes, and not meant to be instanciated by the user.
Arghs. As #18534 got rejected, now there is no way for the user to found is_selfdual()
. And are you sure that category of finite posets is meant to be internal? I know that the class is.
comment:14 in reply to: ↑ 13 ; followup: ↓ 15 Changed 3 years ago by
And are you sure that category of finite posets is meant to be internal? I know that the class is.
I do not think that the category of Posets is meant to be internal.
comment:15 in reply to: ↑ 14 Changed 3 years ago by
Replying to ncohen:
And are you sure that category of finite posets is meant to be internal? I know that the class is.
I do not think that the category of Posets is meant to be internal.
I guess you meant the category of finite posets. Then my patch should be OK (but needs review, of course). And now just force_spring_layout
needs checking.
comment:16 Changed 3 years ago by
Oh, my mistake. I was thinking of the class FinitePoset
(without the terminal 's'). No problem with your last commit.
comment:17 Changed 3 years ago by
 Status changed from needs_review to needs_work
I will change some M\"obius
to Möbius
, so please wait a little more.
comment:18 Changed 3 years ago by
 Commit changed from f43f9c4da4378b56cc0bc5e8cdc234754ceaebc1 to d96eb1ed2c6f58d64f8fb98c4076dd04a686a900
Branch pushed to git repo; I updated commit sha1. New commits:
d96eb1e  From M\"obius to Möbius.

comment:19 Changed 3 years ago by
 Commit changed from d96eb1ed2c6f58d64f8fb98c4076dd04a686a900 to 712ebe89cda1b0e0804a680aac62db60acf9ef7b
Branch pushed to git repo; I updated commit sha1. New commits:
712ebe8  sage.graphs.graph_plot_js.gen_html_code, doc of force_spring_layout.

comment:20 in reply to: ↑ 11 Changed 3 years ago by
 Status changed from needs_work to needs_review
Replying to ncohen:
 What is meant by "(see
GenericGraph
andGenericGraph
)" on documentation of optionforce_spring_layout
?Odd. Well, I guess it was meant to be references toward
.show
and.plot
.
I removed those links. Ready for review.
comment:21 Changed 3 years ago by
Okay.
comment:22 Changed 3 years ago by
Nathann, look at shard_preorder_graph()
on catalog of posets and lattices. The link is broken.
comment:23 followup: ↓ 24 Changed 3 years ago by
On my machine it seems fine.
comment:24 in reply to: ↑ 23 Changed 3 years ago by
Replying to ncohen:
On my machine it seems fine.
Arghs. Partially built documentation. Sorry for noice.
comment:25 Changed 3 years ago by
Beta6 was released, so maybe this one should be closed. Nathann is busy. Frédéric, can you check my changes: 1) "M\"oibus" to "Möbius" is hasse_diagram.py
, 2) two links added in posets/__init__.py
and 3) removed links in graph_plot_js.py
? After these this can be set to postive review, as I already check changes made by you and Nathann.
comment:26 followup: ↓ 27 Changed 3 years ago by
There is no problem with those commits, consider them reviewed.
comment:27 in reply to: ↑ 26 Changed 3 years ago by
 Reviewers changed from Jori Mäntysalo to Jori Mäntysalo, Nathann Cohen
 Status changed from needs_review to positive_review
Replying to ncohen:
There is no problem with those commits, consider them reviewed.
Then everything is reviewed. I added your name to reviewrs list. Thanks.
comment:28 Changed 3 years ago by
 Branch changed from public/ticket/19023 to 712ebe89cda1b0e0804a680aac62db60acf9ef7b
 Resolution set to fixed
 Status changed from positive_review to closed
Hellooooooooo !
I fixed some of the points you listed here, and so I add the commit. To be honest I am a bit scared to add a commit on a ticket that is not in
needs_review
, for I am worried that it may get forgotten... Saw this happen too many times already:/
Nathann
New commits:
trac #19023: Broken graph/ links