Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix phonetics for 切 #578

Merged
merged 1 commit into from
Feb 2, 2025
Merged

Fix phonetics for 切 #578

merged 1 commit into from
Feb 2, 2025

Conversation

xatier
Copy link
Contributor

@xatier xatier commented Jan 31, 2025

The concise dictionary prefers 反切 ㄈㄢˇ ㄑㄧㄝˋ

Add phrses from the concise dictionary.

Ref: https://dict.concised.moe.edu.tw/search.jsp?md=1&word=%E5%88%87#searchL

Summary by Bito

This PR enhances the phonetic mappings for the Chinese character '切' by adding multiple entries from the concise dictionary. The changes include adding Bopomofo phonetic notations for various compound words and phrases, improving the accuracy and completeness of the phonetic system. Some duplicate entries were identified in the mappings that require cleanup.

Unit tests added: False

Estimated effort to review (1-5, lower is better): 2

The concise dictionary prefers `反切 ㄈㄢˇ ㄑㄧㄝˋ`

Add phrses from the concise dictionary.

Ref: https://dict.concised.moe.edu.tw/search.jsp?md=1&word=%E5%88%87#searchL
@Copilot Copilot bot review requested due to automatic review settings January 31, 2025 15:23

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (2)
  • Source/Data/BPMFMappings.txt: Language not supported
  • Source/Data/phrase.occ: Language not supported
Copy link

bito-code-review bot commented Jan 31, 2025

Code Review Agent Run #270c23

Actionable Suggestions - 2
  • Source/Data/BPMFMappings.txt - 2
Review Details
  • Files reviewed - 2 · Commit Range: e8334de..e8334de
    • Source/Data/BPMFMappings.txt
    • Source/Data/phrase.occ
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
    • MyPy (Static Code Analysis) - ✔︎ Successful
    • Astral Ruff (Static Code Analysis) - ✔︎ Successful

AI Code Review powered by Bito Logo

Copy link

Changelist by Bito

This pull request implements the following key changes.

Key Change Files Impacted
Feature Improvement - Enhanced Chinese Phonetic Mappings

BPMFMappings.txt - Added multiple phonetic mappings for '切' character and related phrases

phrase.occ - Added occurrence data for new phrases containing '切' character

@lukhnos lukhnos merged commit d0cc3ef into openvanilla:master Feb 2, 2025
1 check passed
@xatier xatier deleted the qie branch February 2, 2025 15:34
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.

None yet

2 participants