-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Option to give an existing vocab file(.pt or text) to preprocess.py #1346
Conversation
Sorry, I was not clear enough in what we wanted. |
Yes! Will do it! |
looks good, thanks ! |
What about the target vocabulary? |
Oh ok, they are bundled in a single .pt file. |
Well the vocab .pt file contains both src vocab and tgt vocab. |
@lordeddard can you please add a comment in the opt.py to be very specific ? |
Yes! Will do! |
Addresses the issue #1304. Tried to implement the first feature, ability to reuse the existing vocabulary.