Opened 11 years ago

Closed 3 years ago

#10561 closed defect (invalid)

Spelling issues in hamiltonian_cycle()

Reported by: jdemeyer Owned by: mvngu
Priority: trivial Milestone: sage-duplicate/invalid/wontfix
Component: documentation Keywords: spelling
Cc: eviatarbach Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by chapoton)

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

comment:1 Changed 9 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 9 years ago by eviatarbach

  • Cc eviatarbach added

comment:3 Changed 9 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:4 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:5 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:6 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:7 Changed 6 years 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

comment:8 Changed 4 years ago by chapoton

  • Branch public/10561 deleted
  • Commit 28005d7f7eeb64a515ecc6efb6b844a95728e33d deleted

comment:9 Changed 4 years ago by chapoton

  • Description modified (diff)

comment:10 Changed 4 years ago by chapoton

  • Description modified (diff)

comment:11 Changed 4 years ago by chapoton

  • Description modified (diff)

comment:12 Changed 3 years ago by chapoton

  • Milestone changed from sage-7.3 to sage-duplicate/invalid/wontfix
  • Resolution set to invalid
  • Status changed from new to closed

obsolete ticket

Note: See TracTickets for help on using tickets.