-
Notifications
You must be signed in to change notification settings - Fork 269
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
Config mrt_ml_mix seems to be unused #126
Comments
thanks for pointing this out! Yes, this parameter was used in the Theano branch, but hasn't been implemented in the Tensorflow branch. A pull request adding it would be very welcome! You should be able to implement this in mrt_cost() in mrt_utils.py , making use of the translation probabilities that are stored in Line 643 in 5727727
|
Hello, I noticed that the config mrt_ml_mix doesn’t seem to be referenced anywhere outside of the config code. I was wondering if this was by design, or if there is a mrt/mile mixed loss that is yet to be implemented.
I would be interested in using / helping develop the feature if possible.
The text was updated successfully, but these errors were encountered: