Opened 6 years ago

Last modified 3 months ago

#10561 new defect

Spelling issues in hamiltonian_cycle()

Reported by: jdemeyer Owned by: mvngu
Priority: trivial Milestone: sage-7.3
Component: documentation Keywords: spelling
Cc: eviatarbach Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: public/10561 (Commits) Commit: 28005d7f7eeb64a515ecc6efb6b844a95728e33d
Dependencies: Stopgaps:

Description

Reported by eviatarbach in #9698 (relating to the files sage/graphs/generic_graph.py and sage/graphs/generic_graph_pyx.pyx):

  1. 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.
  1. "neccesary" should be "necessary"
  1. "non hamiltonian" should be "non-Hamiltonian" (note the dash)
  1. "Dodecahedral" should not be capitalized; "dodecahedral"
  1. "Running the algorithm on random instances, just to make sure the answers are still satisfiable path". Should be "paths".
  1. "ajacent" should be "adjacent"
  1. "Now, we try the algorithm on a non hamiltonian graph, the Petersen graph" should be "graph: the".
  1. "another known hypohamiltonian graph, the generalized Petersen" should be "graph: the"

Change History (7)

comment:1 Changed 4 years ago by knsam

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

comment:2 Changed 3 years ago by eviatarbach

  • Cc eviatarbach added

comment:3 Changed 3 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:4 Changed 3 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:5 Changed 2 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:6 Changed 2 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:7 Changed 3 months ago by chapoton

  • Branch set to public/10561
  • Commit set to 28005d7f7eeb64a515ecc6efb6b844a95728e33d
  • Milestone changed from sage-6.4 to sage-7.3

New commits:

​28005d7trac 10561 some typos
Note: See TracTickets for help on using tickets.