{"action":"created","issue":{"url":"https://api.github.com/repos/mlpack/mlpack/issues/465","labels_url":"https://api.github.com/repos/mlpack/mlpack/issues/465/labels{/name}","comments_url":"https://api.github.com/repos/mlpack/mlpack/issues/465/comments","events_url":"https://api.github.com/repos/mlpack/mlpack/issues/465/events","html_url":"https://github.com/mlpack/mlpack/pull/465","id":113540328,"number":465,"title":"Create a function to visualize the features learned by sparse autoencoder","user":{"login":"stereomatchingkiss","id":1801914,"avatar_url":"https://avatars.githubusercontent.com/u/1801914?v=3","gravatar_id":"","url":"https://api.github.com/users/stereomatchingkiss","html_url":"https://github.com/stereomatchingkiss","followers_url":"https://api.github.com/users/stereomatchingkiss/followers","following_url":"https://api.github.com/users/stereomatchingkiss/following{/other_user}","gists_url":"https://api.github.com/users/stereomatchingkiss/gists{/gist_id}","starred_url":"https://api.github.com/users/stereomatchingkiss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stereomatchingkiss/subscriptions","organizations_url":"https://api.github.com/users/stereomatchingkiss/orgs","repos_url":"https://api.github.com/users/stereomatchingkiss/repos","events_url":"https://api.github.com/users/stereomatchingkiss/events{/privacy}","received_events_url":"https://api.github.com/users/stereomatchingkiss/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":22,"created_at":"2015-10-27T09:28:33Z","updated_at":"2015-11-23T05:48:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mlpack/mlpack/pulls/465","html_url":"https://github.com/mlpack/mlpack/pull/465","diff_url":"https://github.com/mlpack/mlpack/pull/465.diff","patch_url":"https://github.com/mlpack/mlpack/pull/465.patch"},"body":"Still lack a meaningful unit test, up until now I only compare the results with the one shown at this website(http://deeplearning.stanford.edu/wiki/index.php/Visualizing_a_Trained_Autoencoder)."},"comment":{"url":"https://api.github.com/repos/mlpack/mlpack/issues/comments/158857836","html_url":"https://github.com/mlpack/mlpack/pull/465#issuecomment-158857836","issue_url":"https://api.github.com/repos/mlpack/mlpack/issues/465","id":158857836,"user":{"login":"stereomatchingkiss","id":1801914,"avatar_url":"https://avatars.githubusercontent.com/u/1801914?v=3","gravatar_id":"","url":"https://api.github.com/users/stereomatchingkiss","html_url":"https://github.com/stereomatchingkiss","followers_url":"https://api.github.com/users/stereomatchingkiss/followers","following_url":"https://api.github.com/users/stereomatchingkiss/following{/other_user}","gists_url":"https://api.github.com/users/stereomatchingkiss/gists{/gist_id}","starred_url":"https://api.github.com/users/stereomatchingkiss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stereomatchingkiss/subscriptions","organizations_url":"https://api.github.com/users/stereomatchingkiss/orgs","repos_url":"https://api.github.com/users/stereomatchingkiss/repos","events_url":"https://api.github.com/users/stereomatchingkiss/events{/privacy}","received_events_url":"https://api.github.com/users/stereomatchingkiss/received_events","type":"User","site_admin":false},"created_at":"2015-11-23T05:48:27Z","updated_at":"2015-11-23T05:48:27Z","body":"The function could do what you suggest, adding two more parameters could make it become more general, but it would make this class more complicated too(more parameter to read and specify). In most cases, I would bet the users only want to use this class to visualize the results of autoencoder and prefer not to change the block height and block width.\r\n\r\n>I'd suggest using size_t instead of arma::uword\r\n\r\nI have no problem with this one"}}