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 露 #577

Merged
merged 1 commit into from
Feb 2, 2025
Merged

Fix phonetics for 露 #577

merged 1 commit into from
Feb 2, 2025

Conversation

xatier
Copy link
Contributor

@xatier xatier commented Jan 31, 2025

Add 露 ㄌㄨˋ variant from concise dictionary preferences.

露馬腳 ㄌㄡˋ ㄇㄚˇ ㄐㄧㄠˇ is the only 露 ㄌㄡˋ been added.

Ref: https://dict.concised.moe.edu.tw/search.jsp?md=1&word=%E9%9C%B2#searchL

Summary by Bito

Enhancement of phonetic mapping system by adding variant pronunciation '˜ㄨˋ' for the character '露' and its compound words. Updates include new entries in BPMFMappings.txt for compound words containing '露' with their phonetic notations, and corresponding occurrence data updates in phrase.occ file.

Unit tests added: False

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

Add `露 ㄌㄨˋ` variant from concise dictionary preferences.

`露馬腳 ㄌㄡˋ ㄇㄚˇ ㄐㄧㄠˇ` is the only `露 ㄌㄡˋ` been added.

Ref: https://dict.concised.moe.edu.tw/search.jsp?md=1&word=%E9%9C%B2#searchL
@Copilot Copilot bot review requested due to automatic review settings January 31, 2025 15:12

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 #77950a

Actionable Suggestions - 0
Review Details
  • Files reviewed - 2 · Commit Range: 8dfe7f1..8dfe7f1
    • 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 - Chinese Phonetic Mapping Enhancement

BPMFMappings.txt - Added multiple phonetic mappings for '露' character and related compound words

phrase.occ - Added occurrence data for new compound words containing '露' character

@lukhnos lukhnos merged commit 930cde3 into openvanilla:master Feb 2, 2025
1 check passed
@xatier xatier deleted the lu 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.

2 participants