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
330 stars 37 forks source link

fixed bug that prevented "download all" functionality #474

Open bunburya opened 1 week ago

bunburya commented 1 week ago

Aggregator.models wasn't passing the value for filter_downloaded through to the models method of the Repository subclass. This means that by default the subclass's models method always fired with filter_downloaded=True, which meant that if there were no models already downloaded, calling bergamot download didn't download anything as it couldn't discover the undownloaded models. This PR fixes this behaviour.