Skip to content

Fix Turkish typos, missing comma, and diacritics in examples.py#14002

Open
omer-said-bayhan wants to merge 2 commits into
explosion:masterfrom
omer-said-bayhan:master
Open

Fix Turkish typos, missing comma, and diacritics in examples.py#14002
omer-said-bayhan wants to merge 2 commits into
explosion:masterfrom
omer-said-bayhan:master

Conversation

@omer-said-bayhan

Copy link
Copy Markdown

Hi, this is a minor fix for the Turkish example sentences to ensure correct string separation and proper use of Turkish-specific characters.

Changes made:

  • Added a missing comma after the "San Francisco..." sentence to prevent Python from automatically concatenating it with the "Londra..." sentence.
  • Fixed missing or incorrect Turkish diacritics (ı, İ, ü) in words like "Bunları" and "Türkiye".
  • Corrected the name of the famous Turkish poet from "Cemal Sureya" to its proper spelling: "Cemal Süreya".

(Türkçe Açıklama)
Merhaba, bu PR Türkçe örnek cümlelerdeki küçük hataları düzeltmek, cümlelerin doğru ayrılmasını ve Türkçeye özgü harflerin (dır/dir ekleri, büyük İ, ü harfi) doğru kullanılmasını sağlamak için açılmıştır.

Yapılan Değişiklikler:

  • "San Francisco..." cümlesinin sonuna eksik virgül eklendi (Python'un bu iki cümleyi tek bir string gibi birleştirmesi engellendi).
  • "Bunları" ve "Türkiye" gibi kelimelerdeki Türkçeye özgü harflerin (ı, İ) eksik kullanımı düzeltildi.
  • Ünlü Türk şairinin ismi "Cemal Sureya" yerine doğru yazılışı olan "Cemal Süreya" olarak güncellendi.

Thanks for the great work on spaCy! / spaCy'deki harika çalışmalar için teşekkürler!

@rkfshakti rkfshakti left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch on the Turkish diacritics and missing commas. The changes are correct:

  • "Cemal Sureya" → "Cemal Süreya" (correct name spelling)
  • "Bunlari" → "Bunları" (correct Turkish dotted i)
  • "Altinoluk" → "Altınoluk", "Turkiye" → "Türkiye", "haritasinin" → "haritasının" (correct Turkish character usage)
  • Missing trailing commas added for consistency

The new tokenizer exceptions (cm, gb, kat, kg, km, mb, da) are useful additions for Turkish text processing. The formatting comments (dashed lines) help readability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants