Skip to content

Conversation

@Carmezim
Copy link
Contributor

An addition to #9882

@tensorflow-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@nealwu
Copy link
Contributor

nealwu commented May 15, 2017

Hi @Carmezim, is there any reason why your link points to a particular commit? By default I would want it to point to https://github.com/tensorflow/models/blob/master/tutorials/embedding/word2vec.py.

@Carmezim
Copy link
Contributor Author

Hi @nealwu. I am so sorry. I was in the wrong branch, you are absolutely right. I wanted to point to r1.1. I fixed to point to master. Thanks for catching it!

@Carmezim Carmezim closed this May 15, 2017
@Carmezim Carmezim reopened this May 15, 2017
@tensorflow-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@nealwu
Copy link
Contributor

nealwu commented May 15, 2017

Looks good!

@rmlarsen rmlarsen merged commit 524a890 into tensorflow:r1.1 May 15, 2017
nealwu added a commit that referenced this pull request May 18, 2017
@nealwu nealwu mentioned this pull request May 18, 2017
rmlarsen pushed a commit that referenced this pull request May 18, 2017
* Revert "fixing broken code repository links"

This reverts commit 84c11ce.

* Revert "[Docs] Fixing old model link (#9886)"

This reverts commit 524a890.
allenlavoie pushed a commit to allenlavoie/tensorflow that referenced this pull request Jul 15, 2017
* Fixing old model link

* Fixing misguided link.
allenlavoie pushed a commit to allenlavoie/tensorflow that referenced this pull request Jul 15, 2017
* Revert "fixing broken code repository links"

This reverts commit 84c11ce.

* Revert "[Docs] Fixing old model link (tensorflow#9886)"

This reverts commit 524a890.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants