browsermt / bergamot-translator

Cross platform C++ library focusing on optimized machine translation on the consumer-grade device.
http://browser.mt
Mozilla Public License 2.0
327 stars 36 forks source link

MacOS Wheels #432

Closed graemenail closed 2 years ago

graemenail commented 2 years ago

Adds extra python wheels for MacOS

graemenail commented 2 years ago

Some portion of the docs requires the bergamot package to be installed. The previous method installed from the wheel directly from cp37-*.

Prior to this PR that was fine, because only the manylinux wheel matched this glob. Now we have a wheel for MacOS matching it too.

I tried to modify the install such that it would pickup the correct wheel automatically. The last commit here removed --no-index because we need to search the index for bergamot dependencies. Perhaps there's a better way..