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

Pitch not shown for all words #642

Closed
mvysny opened this issue Mar 15, 2016 · 9 comments
Closed

Pitch not shown for all words #642

mvysny opened this issue Mar 15, 2016 · 9 comments
Labels
bug

Comments

@mvysny
Copy link
Owner

@mvysny mvysny commented Mar 15, 2016

For example higashi 東

@mvysny mvysny added the bug label Mar 15, 2016
@Chinchilla-white
Copy link

@Chinchilla-white Chinchilla-white commented Mar 15, 2016

Another example: 出口

@Chinchilla-white
Copy link

@Chinchilla-white Chinchilla-white commented Mar 15, 2016

Wrong :入り口

@mvysny
Copy link
Owner Author

@mvysny mvysny commented Mar 15, 2016

Thank you for letting me know. Which reading is wrong for 入り口?

@mvysny
Copy link
Owner Author

@mvysny mvysny commented Mar 15, 2016

Fixed, please update JMDict.

@mvysny mvysny closed this Mar 15, 2016
@Chinchilla-white
Copy link

@Chinchilla-white Chinchilla-white commented Mar 15, 2016

What is the little circle between ぐ and ち ?
There is also a little circle between が and し in ひがし.
It seems that the little circle appears after a hiragana with ten ten and with a high pitch (so it has a line above it).

@mvysny
Copy link
Owner Author

@mvysny mvysny commented Mar 15, 2016

The only information about that little circle (°) I have is that it is called "nasal". Perhaps this one? https://en.wikipedia.org/wiki/Dakuten

@Chinchilla-white
Copy link

@Chinchilla-white Chinchilla-white commented Mar 15, 2016

I see. I remember now there's a column 'nasal' in the DB. Could you add an option to hide the "nasal" sound or simply remove this "nasal" circle altogether? This "nasal" sound is pronounced by NHK presenters as standard Tokyo accent. But it's also considered outdated by some Japanese. Thanks a lot!

@mvysny
Copy link
Owner Author

@mvysny mvysny commented Mar 15, 2016

Yup, that's the one. Sure, I'll add a configuration option for that.

@mvysny
Copy link
Owner Author

@mvysny mvysny commented Mar 16, 2016

Closing; the configuration option will be added as a fix for #644

@mvysny mvysny closed this Mar 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.