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 呱 #586

Merged
merged 1 commit into from
Feb 8, 2025
Merged

Fix phonetics for 呱 #586

merged 1 commit into from
Feb 8, 2025

Conversation

xatier
Copy link
Contributor

@xatier xatier commented Feb 7, 2025

The concise dictionary prefers 呱 ㄍㄨ.

The revised dictionary prefers 呱 ㄨㄚ for the following:

呱呱 ㄨㄚ ㄨㄚ
呱呱墮地 ㄨㄚ ㄨㄚ ㄉㄨㄛˋ ㄉㄧˋ
呱呱墜地 ㄨㄚ ㄨㄚ ㄓㄨㄟˋ ㄉㄧˋ

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

Summary by Bito

This pull request introduces phonetic updates for the character '呱', including new mappings in BPMFMappings.txt and an updated entry in phrase.occ. These changes enhance the accuracy of phonetic representations in the application.

Unit tests added: False

Estimated effort to review (1-5, lower is better): 2 - The changes are straightforward and primarily involve updates to existing files, making the review process relatively simple.

The concise dictionary prefers `呱 ㄍㄨ`.

The revised dictionary prefers `呱 ㄨㄚ` for the following:

```text
呱呱 ㄨㄚ ㄨㄚ
呱呱墮地 ㄨㄚ ㄨㄚ ㄉㄨㄛˋ ㄉㄧˋ
呱呱墜地 ㄨㄚ ㄨㄚ ㄓㄨㄟˋ ㄉㄧˋ
```

Ref:
https://dict.concised.moe.edu.tw/search.jsp?md=1&word=%E5%91%B1#searchL
https://dict.revised.moe.edu.tw/search.jsp?md=1&word=%E5%91%B1
@Copilot Copilot bot review requested due to automatic review settings February 7, 2025 13:29
Copy link

@Copilot Copilot AI left a comment

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

@lukhnos lukhnos merged commit a1b82a2 into openvanilla:master Feb 8, 2025
2 checks passed
@xatier xatier deleted the gu branch February 8, 2025 13:13
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