search

m-rtin / anki-ipa

Anki add-on to insert IPA transcriptions
https://ankiweb.net/shared/info/799647424
GNU Affero General Public License v3.0
21 stars 12 forks source link

Fix for some languages IPA not populating #64

Closed syntheticgio closed 1 year ago

syntheticgio commented 1 year ago

Made a few changes that might be desirable to merge into the master branch:

  1. Fix for Spanish not populating properly
  2. Added a config.md with descriptions on the config options
  3. Added a config option to be able to maintain the '.' in IPA. Default is same behavior it currently is
  4. Added language options: italian, portuguese, chinese (simplified), and catalan.
peter-moran commented 1 year ago

How can I package this to try in Anki?

soliviantar commented 1 year ago

@peter-moran @syntheticgio I tried installing this by downloading the zip file from here and installing the add-on manually, following these instructions. However, it didn't work. It threw this error:


Error
An add-on you installed failed to load. If problems persist, please go to the Tools>Add-ons menu, and disable or delete the add-on.

When loading '⁨anki_ipa⁩':
⁨Traceback (most recent call last):
  File "aqt.addons", line 246, in loadAddons
  File "C:\Users\g\AppData\Roaming\Anki2\addons21\anki_ipa\__init__.py", line 60
    ipa = parse_ipa_transcription.transcript(words=words, language=lang_alias, CONFIG["STRIP_SYLLABLE_SEPARATOR"])
                                                                                                                 ^
SyntaxError: positional argument follows keyword argument
m-rtin commented 1 year ago

I updated the add-on on Anki web with the changes you made. I removed the languages you added again though since they were not working properly. Feel free to add them again in another PR with proper unit-testing :)