Skip to content
This repository has been archived by the owner on Dec 11, 2023. It is now read-only.

Is this repo deprecated? #465

Open
nashid opened this issue Apr 3, 2020 · 4 comments
Open

Is this repo deprecated? #465

nashid opened this issue Apr 3, 2020 · 4 comments

Comments

@nashid
Copy link

nashid commented Apr 3, 2020

There is no activity in this repo.

What would be better resource for building NMT models?

@mommi84
Copy link

mommi84 commented May 21, 2020

@luozhouyang
Copy link

This implementation is too old. You can implement NMT with tf 2.0 easily. As @mommi84 mentioned, nmt_with_attention is an excellent tutorial.

If you want a Transformer model, you can find it in my repo: transformers-keras

@nashid
Copy link
Author

nashid commented Jun 8, 2020

Thanks @luozhouyang.

I will definitely have a look at the nmt_with_attention tutorial.

Actually, at first I was thinking to use the model from that tutorial. However, I was not sure how that will trade as this repo seems to be authoritative and many scientific publications have cited this repo.

If you have any pointer related to this, do let me know.

I will look into the tutorial and would a comparison.

@mommi84
Copy link

mommi84 commented Jun 9, 2020

You're welcome, @nashid. 😅

I think the reason this repo seems authoritative is simply that it had been pioneering in the sequence-to-sequence era. It could be that today the accuracy you obtain will be higher than the new models', but likely not in the long term once tf 2.0 reaches the plateau of productivity.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants