Conversation
|
Works on my browser 👍 I agree this isn't a good solution for small visualizations. Blowing everything up to some minimum size might work. What about a pop-out like the "Basic Zoom" on this page? We want it to work and look good on big/small images and big/small browser windows. |
|
I tried the "Basic/Mousewheel Zoom" scheme from the Elevateweb demo page however if the visualisation image is tiny in the first place it is hardly possible to control anything using the mouse. I have added a patch to upscale the visualisation image in case it is less than 200 pixels in height. This looked slightly better to me although one possible downside is all small visualisations would have the same image size so for example the effect of pooling layers is less visually obvious. |
|
Where did you get the |
|
Yeah, this doesn't seem like a good solution for data with shape If you leave the image small and change the CSS for your img to be: <img style="max-width:100%;min-width:50px;image-rendering:pixelated;" />I like that better, what do you think? Still playing around with the zoom options ... |
|
Augh, I can't get this to work with |
54dd7c3 to
3c62722
Compare
|
There was a bug in elevate zoom which caused zooming to fail on square images. I fixed it on elevateweb/elevatezoom#115 but the project seems to be frozen. How do you recommend I shall mention the origin of the file when adding it to the DIGITS github? The |
We need the copyright header as the author intended it, even if we make a small change. Start with this file or this file, make the fix, minimize it, and then copy the header from here or here, depending on which version you started with.
Yeah, I was afraid of that. Oh well.
Hah! Figures. |
|
Oh yeah, this is starting to look good! Zooming in on gabor filters in Upscaling for small fully-connected layers: @gheinrich I like this a lot, and it's working for me on both Chrome and Firefox. Anything else you want to change before merging? (Besides the copyright) |
3c62722 to
d6ac055
Compare
|
Ah, the javascript "minification" script I used removed all comments including the copyright. But my starting point was this file, as you suggested. I have squashed the commits and uploaded the Pull Request. I hope it's OK now. |
d6ac055 to
80e3330
Compare
|
No problem, I have updated the patch. Which network is this by the way? Interesting to see non-square filters (?) |
|
Awesome, I'll merge as soon as TravisCI passes. Coveralls is having problems - ignore the stalled checks.
I just made a little noop network to generate activations with the size I wanted to test 😏. |






This appears to be working on popular browsers.
Caveat: zooming on 'small' layers (those that have very few weights/activations) feels slightly awkward. Perhaps we should set a minimum size for visualisation images e.g. 256x256?