-
Notifications
You must be signed in to change notification settings - Fork 257
Support loading fasttext model from custom file #102
base: master
Are you sure you want to change the base?
Conversation
Related to issue 61 |
Codecov Report
@@ Coverage Diff @@
## master #102 +/- ##
=======================================
Coverage 94.41% 94.42%
=======================================
Files 64 64
Lines 1611 1613 +2
=======================================
+ Hits 1521 1523 +2
Misses 90 90
Continue to review full report at Codecov.
|
Hi! Thanks for your contribution. It looks like the parameters introduced are directly passed into |
Yes, this is a good idea. It is more suitable for the problem. I didn't understand the related class thoroughly before. |
Would you like to change |
How do I load this in pytorch or pytorch-nlp Is this feature completed or still in progress ? |
If I understand correctly, it should be done by just passing the name of your file to the "name" parameter. However, the suggestion from PetrochukM is more appropriate, if you have time you could follow his suggestion and fix the issue. Thanks! |
Add two simple parameters to class FastText, making it possible to load fasttext model from custom file.