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

Update the embedding.md file #473

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
3 participants
@amanp592
Copy link
Contributor

commented Apr 9, 2019

Solves the issue as mentioned in the tensorflow repository #27612

Update the embedding.md file
Solves the issue as mentioned in #27612

@amanp592 amanp592 requested review from lamberta and MarkDaoust as code owners Apr 9, 2019

@googlebot googlebot added the cla: yes label Apr 9, 2019

@amanp592

This comment has been minimized.

Copy link
Contributor Author

commented Apr 9, 2019

@lamberta @MarkDaoust kindly review this PR


To use images as metadata, you must produce a single
[sprite image](https://www.google.com/webhp#q=what+is+a+sprite+image),
consisting of small thumbnails, one for each vector in the embedding. The
consisting of small thumbnails, one for each vector in the embedding.
This single image contain all the images the dataset has.The

This comment has been minimized.

Copy link
@lamberta

lamberta Apr 9, 2019

Member

This single image contain all the images the dataset has.

Can you clarify this?

This comment has been minimized.

Copy link
@amanp592

amanp592 Apr 9, 2019

Author Contributor

@lamberta yeah i did something wrong there, i was suppose to write that a sprite image is a collection of all sample images into one single image.

format.
format.Also, when you open the load data tab in the embedding visualizer there
it is specified about what is TSV file of vector and what is the TSV file
of metadata with an example.

This comment has been minimized.

Copy link
@lamberta

lamberta Apr 9, 2019

Member

Clean up. Is this accurate? ...
The embedding visualizer's Load Data tab specifies the TSV file for the vector and metadata.

This comment has been minimized.

Copy link
@amanp592

amanp592 Apr 9, 2019

Author Contributor

i'll correct this

@amanp592 amanp592 referenced this pull request Apr 9, 2019

Merged

Updated embedding.md #474

@lamberta

This comment has been minimized.

Copy link
Member

commented Apr 22, 2019

This change was moved to #474

@lamberta lamberta closed this Apr 22, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.