Skip to content
This repository was archived by the owner on Jul 7, 2023. It is now read-only.

Conversation

Styleoshin
Copy link
Contributor

This is a little tutorial about how to use tensor2tensor for translation task. Because in first time its hard to understand how this library works.
I had to use this library for a project, I was wondering why not create a little tutorial at the same time, for help others.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@review-notebook-app
Copy link

Check out this pull request on ReviewNB: https://app.reviewnb.com/tensorflow/tensor2tensor/pull/1675

You'll be able to see notebook diffs and discuss changes. Powered by ReviewNB.

@googlebot googlebot added the cla: no PR author has not signed CLA label Aug 23, 2019
@lukaszkaiser
Copy link
Contributor

Thanks! Could you click the CLA part? (We need this to accept the PR - let us know if there're any problems with that.)

@Styleoshin
Copy link
Contributor Author

@googlebot I signed it!

@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@Styleoshin
Copy link
Contributor Author

@googlebot I fixed it.

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: yes PR author has signed CLA and removed cla: no PR author has not signed CLA labels Aug 26, 2019
@afrozenator
Copy link
Contributor

Hi @Styleoshin -- First of all many thanks for volunteering to add this, we'd like to accept this.

But we require an Apache license at the start of all the files, will you be able to add that (see the other notebooks in this directory for an example)?

I can merge it in once you do that, all else looks in place.

Thanks

@Styleoshin
Copy link
Contributor Author

Hi @afrozenator! So much.
I added the license.

@afrozenator
Copy link
Contributor

Thanks @Styleoshin ! Will merge in shortly.

@afrozenator afrozenator merged commit 6f4f72f into tensorflow:master Aug 26, 2019
tensorflow-copybara pushed a commit that referenced this pull request Aug 26, 2019
PiperOrigin-RevId: 265526105
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes PR author has signed CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants