Opened 5 years ago

Last modified 22 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:


Reported by eviatarbach in #9698 (relating to the files sage/graphs/ 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: "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 (6)

comment:1 Changed 3 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?


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 2 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 22 months ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4
Note: See TracTickets for help on using tickets.