Opened 4 years ago

Last modified 6 weeks 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):

  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 (6)

comment:1 Changed 19 months 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 18 months ago by eviatarbach

  • Cc eviatarbach added

comment:3 Changed 13 months ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:4 Changed 8 months ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:5 Changed 5 months ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:6 Changed 6 weeks ago by vbraun_spam

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