search

accraze / machine-translator

Translate words using a statistical model
https://www.npmjs.com/package/machine-translator
MIT License
8 stars 7 forks source link

dist/translator.js is missing #130

Open alexanderby opened 4 years ago

alexanderby commented 4 years ago

Bundles are not distributed.

accraze commented 4 years ago

Hey thanks for the bug report @alexanderby . I'll get a patch submitted in the near future.

alexanderby commented 4 years ago

I looked through the package and found many bugs. Has it ever worked?

accraze commented 4 years ago

Yikes, yeah looks like there's been some issues since v0.8.2 when ES6 support was added. I haven't really been maintaining this package for the last couple of years, although I have some interest in doing so now.

I'll try to find some time to get this package back in working order. If you have any suggestions or quick fixes, feel free to submit a PR 😸

alexanderby commented 4 years ago

There are issues like this.transmissions.n.f instead of this.transmissions[native][foreign]. Dependency package doesn't support UTF-8, but ASCII is hardcoded instead. Also I remember some code branches were unused.

I used another approach for my problem, so there is no critical need in your package at the moment. But if it worked it could be helpful, because I see no other JS translation examples and still would like to know how this algorithm works.