distillpub / post--feature-wise-transformations

Feature-Wise Transformations
https://distill.pub/2018/feature-wise-transformations
Creative Commons Attribution 4.0 International
20 stars 11 forks source link

Add hovering feature to t-SNE plots #169

Closed vdumoulin closed 6 years ago

fstrub95 commented 6 years ago

https://github.com/vdumoulin/distill-film/pull/175

fstrub95 commented 6 years ago

The clusters are actually a bit blur, what perplexity did you use? I would recommend using a perplexity between 200 and 300 while computing the tsne.

vdumoulin commented 6 years ago

I don't recall, I'll doublecheck to be sure.

vdumoulin commented 6 years ago

@fstrub95 I added the questions for CLEVR. The perplexity I'm using is 350. One thing that's puzzling me is that question types don't seem to match the questions themselves.

fstrub95 commented 6 years ago

I will look at it. If there is a bug, it is likely due to the hard-coded javascript list of question types instead of extracting one from the json file (like in artists)

On Mon, Jun 11, 2018 at 3:29 PM, vdumoulin notifications@github.com wrote:

@fstrub95 https://github.com/fstrub95 I added the questions for CLEVR. The perplexity I'm using is 350. One thing that's puzzling me is that question types don't seem to match the questions themselves.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vdumoulin/distill-film/issues/169#issuecomment-396263421, or mute the thread https://github.com/notifications/unsubscribe-auth/AMvzhAq1ZfAlPfRCY0P1JUZyJSQZR2Wmks5t7n6ugaJpZM4UgZAe .

vdumoulin commented 6 years ago

Sounds good! FYI, I pushed the style images as well, so the two plots now have the correct tooltips :).