Skip to content
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

Embedding visualization #351

Merged
merged 7 commits into from
Mar 30, 2018

Conversation

jetfuel
Copy link
Collaborator

@jetfuel jetfuel commented Mar 29, 2018

Backend:
Create the embedding log writer and reader.

User can add embeddings by supplying a words vectors and a label list.

Create the embedding logger and reader.
User can now save a list of word vectors along with a list of labels.
@jetfuel jetfuel self-assigned this Mar 29, 2018
}

// Add all word vectors along with all labels
// The index of lables should match with the index of word_embeddings
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

labels.

@@ -109,6 +109,7 @@ message Tablet {
kImage = 2;
kText = 3;
kAudio = 4;
kkEmbedding = 5;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be 'kEmbedding'

@daming-lu daming-lu merged commit a8cadfb into PaddlePaddle:develop Mar 30, 2018
@jetfuel jetfuel deleted the embedding_visualization branch March 30, 2018 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants