python / python/cpython

Wrong lemma in two_way_notes.txt

Offen
#132,088 6 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

docs
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
35.9k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

Hi @sweeneyde @tim-one

I think there is a wrong claim in the notes about two-way algorithm.

https://github.com/python/cpython/blob/255eb375a7cc3cb16698bebf42689e8bc783ed93/Objects/stringlib/stringlib_find_two_way_notes.txt#L352-L353

Let s = (abcccd)^2 = abcccdabcccd and p = ccdabcc of length len(p) = 7 and period r = 5.

The period of the whole string s is 6, which is strictly greater than r but strictly smaller than len(p).

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Lies in Objects/stringlib/stringlib_find_two_way_notes.txt etwa die Zeilen 352–353 und vergleiche das dort angegebene Lemma mit dem Gegenbeispiel im Issue. Prüfe die umgebende Diskussion über Perioden in den Notizen zum Two-Way-Algorithmus, korrigiere dann die falsche Behauptung und verifiziere, dass das Beispiel konsistent behandelt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
documentation
Issue-Typ
Dokumentation
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.