Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

Update database.py #175

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Update database.py #175

wants to merge 1 commit into from

Conversation

ahj10
Copy link

@ahj10 ahj10 commented Jan 24, 2021

Added the same code you have going on for Mandarin Pinyin to Cantonese Jyutping. The database cidian is lacking almost all the jyutping so there is no way it will work as of your current status. So I ended up consulting your other database, hanzi kCantonese, for all my Cantonese queries. Since kCantonese has different pronunciations for the majority of characters, I just decided to pick the first one for ease of reading in the anki card. Finally, I had to erase the ~100 jyutping you had in your cidian database so it wouldn't conflict.

Added the same code you have going on for Mandarin Pinyin to Cantonese Jyutping. The database cidian is lacking almost all the jyutping so there is no way it will work as of your current status. So I ended up consulting your other database, hanzi kCantonese, for all my Cantonese queries. Since kCantonese has different pronunciations for the majority of characters, I just decided to pick the first one for ease of reading in the anki card. Finally, I had to erase the ~100 jyutping you had in your cidian database so it wouldn't conflict.
@ahj10
Copy link
Author

ahj10 commented Jan 24, 2021

Added the same code you have going on for Mandarin Pinyin to Cantonese Jyutping. The database cidian is lacking almost all the jyutping so there is no way it will work as of your current status. So I ended up consulting your other database, hanzi kCantonese, for all my Cantonese queries. Since kCantonese has different pronunciations for the majority of characters, I just decided to pick the first one for ease of reading in the anki card. Finally, I had to erase the ~100 jyutping you had in your cidian database so it wouldn't conflict.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant