Opened 4 years ago
Last modified 8 months ago
#10561 new defect
Spelling issues in hamiltonian_cycle()
Reported by: | jdemeyer | Owned by: | mvngu |
---|---|---|---|
Priority: | trivial | Milestone: | sage-6.4 |
Component: | documentation | Keywords: | spelling |
Cc: | eviatarbach | Merged in: | |
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
Reported by eviatarbach in #9698 (relating to the files sage/graphs/generic_graph.py and sage/graphs/generic_graph_pyx.pyx):
- All instances of "hamiltonian" should be changed to "Hamiltonian". This is because Hamilton is a proper noun, and the capitalization transfers to the term (see here: http://mathworld.wolfram.com/HamiltonianCycle.html). "hypohamiltonian" should be left lower-case, though.
- "neccesary" should be "necessary"
- "non hamiltonian" should be "non-Hamiltonian" (note the dash)
- "Dodecahedral" should not be capitalized; "dodecahedral"
- "Running the algorithm on random instances, just to make sure the answers are still satisfiable path". Should be "paths".
- "ajacent" should be "adjacent"
- "Now, we try the algorithm on a non hamiltonian graph, the Petersen graph" should be "graph: the".
- "another known hypohamiltonian graph, the generalized Petersen" should be "graph: the"
Change History (6)
comment:1 Changed 2 years ago by knsam
comment:2 Changed 2 years ago by eviatarbach
- Cc eviatarbach added
comment:3 Changed 20 months ago by jdemeyer
- Milestone changed from sage-5.11 to sage-5.12
comment:4 Changed 15 months ago by vbraun_spam
- Milestone changed from sage-6.1 to sage-6.2
comment:5 Changed 12 months ago by vbraun_spam
- Milestone changed from sage-6.2 to sage-6.3
comment:6 Changed 8 months ago by vbraun_spam
- Milestone changed from sage-6.3 to sage-6.4
Note: See
TracTickets for help on using
tickets.
Let me just confirm the work issues: we don't want the changes to affect the function names, right? Should this change affect the error messages?
~KnS