search

bartvm / nmt

Neural machine translation
MIT License
2 stars 2 forks source link

Squashed multi-GPU without unknown context embeddings #25

Closed bartvm closed 8 years ago

bartvm commented 8 years ago

There's still a lot of duplicated code between nmt_single.py and nmt_worker.py that I'll try to get rid off.

Runs only on Python 2 because of Platoon.

bartvm commented 8 years ago

@anirudh9119 Let me know if this looks okay for you; it's basically just #24 rebased, with unknown context embeddings removed, and some Python 3 fixes applied to nmt_worker.py

anirudh9119 commented 8 years ago

Okay!

nouiz commented 8 years ago

platton was updated for python 3 I think.

On Mon, Feb 8, 2016 at 2:56 PM, Bart van Merriënboer < notifications@github.com> wrote:

Merged #25 https://github.com/bartvm/nmt/pull/25.

— Reply to this email directly or view it on GitHub https://github.com/bartvm/nmt/pull/25#event-543282845.