-
Notifications
You must be signed in to change notification settings - Fork 9
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
complete adding download button #43
base: main
Are you sure you want to change the base?
complete adding download button #43
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! I'm unsure whether we need this functionality for the Anonymous and Side-by-side windows but maybe only for the Single window? @sivareddyg maybe you could chime in as you requested this feature I think?
Co-authored-by: Niklas Muennighoff <[email protected]>
Co-authored-by: Niklas Muennighoff <[email protected]>
Co-authored-by: Niklas Muennighoff <[email protected]>
Co-authored-by: Niklas Muennighoff <[email protected]>
@Muennighoff addressed PR review comments |
@SaitejaUtpala This is great, thank you! can I see a live preview of the page with these modifications? Downloading the embedding is one thing, but it would also be useful to provide some helper code on how to use the resulting embedding. Can you also provide an example on how to use the resulting embedding to compute cosine similarity and plot them in a TSNE plot. This could be in the form of toggle button. This would attract people to come to arena to do other things. For example, many times, I want just download a few embeddings and play with them myself but without having to run the actual models on my machine, and this would make it super easy. |
@sivareddyg On it! |
This PR download buttons for (#42 )
some comments