{"action":"created","issue":{"url":"https://api.github.com/repos/SciSpark/SciSpark/issues/72","repository_url":"https://api.github.com/repos/SciSpark/SciSpark","labels_url":"https://api.github.com/repos/SciSpark/SciSpark/issues/72/labels{/name}","comments_url":"https://api.github.com/repos/SciSpark/SciSpark/issues/72/comments","events_url":"https://api.github.com/repos/SciSpark/SciSpark/issues/72/events","html_url":"https://github.com/SciSpark/SciSpark/issues/72","id":163942236,"number":72,"title":"Reshape operation for SciTensor","user":{"login":"rahulpalamuttam","id":5025939,"avatar_url":"https://avatars.githubusercontent.com/u/5025939?v=3","gravatar_id":"","url":"https://api.github.com/users/rahulpalamuttam","html_url":"https://github.com/rahulpalamuttam","followers_url":"https://api.github.com/users/rahulpalamuttam/followers","following_url":"https://api.github.com/users/rahulpalamuttam/following{/other_user}","gists_url":"https://api.github.com/users/rahulpalamuttam/gists{/gist_id}","starred_url":"https://api.github.com/users/rahulpalamuttam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rahulpalamuttam/subscriptions","organizations_url":"https://api.github.com/users/rahulpalamuttam/orgs","repos_url":"https://api.github.com/users/rahulpalamuttam/repos","events_url":"https://api.github.com/users/rahulpalamuttam/events{/privacy}","received_events_url":"https://api.github.com/users/rahulpalamuttam/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/SciSpark/SciSpark/labels/ESIP%20workshop","name":"ESIP workshop","color":"fef2c0"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-07-05T20:43:34Z","updated_at":"2016-07-05T21:53:09Z","closed_at":null,"body":"I've added this issue for the ESIP workshop, as I'm sure several users will be interested in reshaping their arrays. I'm seeing the need as I am attempting to replicate @agoodm succesful attempt at PDF clustering using the standard deviation and skewness. The netcdf file he has given me has a tasjan variable shaped (1023, 181, 286). I want to deal with the array with shape (33, 31, 181, 286).\r\n\r\nThe api call will look like this :\r\nsciT(\"var\").reshape(33, 31, 181, 286)\r\n\r\nHere are a few questions/points I wanted to brainstorm on :\r\n\r\n1. I originally thought it would be a good idea for AbstractTensor to also implement reshape. However Breeze only supports two dimensions. Nd4j on the otherhand. Nd4j on the otherhand does support reshaping. This is done by using the constructor and a new shape parameter.  \r\nThus for now it makes sense to only provide the reshape function in SciTensor and not in the other tensor classes.\r\n\r\n2. The reshape operation will be applied on the variable in use in the SciTensor. \r\nDo we want the reshaped AbstractTensor to exist in another SciTensor, replace the corresponding AbstractTensor in the variable hashmap, or insert it as a new variable all together (the name is specified as an extra parameter by the user. \r\nThe api call will look like this:\r\n\r\nsciT(\"var\").reshape(\"reshapedVar\", 33, 31, 181, 286)\r\n\r\n@BrianWilson1 @kwhitehall thoughts?\r\n"},"comment":{"url":"https://api.github.com/repos/SciSpark/SciSpark/issues/comments/230614232","html_url":"https://github.com/SciSpark/SciSpark/issues/72#issuecomment-230614232","issue_url":"https://api.github.com/repos/SciSpark/SciSpark/issues/72","id":230614232,"user":{"login":"rahulpalamuttam","id":5025939,"avatar_url":"https://avatars.githubusercontent.com/u/5025939?v=3","gravatar_id":"","url":"https://api.github.com/users/rahulpalamuttam","html_url":"https://github.com/rahulpalamuttam","followers_url":"https://api.github.com/users/rahulpalamuttam/followers","following_url":"https://api.github.com/users/rahulpalamuttam/following{/other_user}","gists_url":"https://api.github.com/users/rahulpalamuttam/gists{/gist_id}","starred_url":"https://api.github.com/users/rahulpalamuttam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rahulpalamuttam/subscriptions","organizations_url":"https://api.github.com/users/rahulpalamuttam/orgs","repos_url":"https://api.github.com/users/rahulpalamuttam/repos","events_url":"https://api.github.com/users/rahulpalamuttam/events{/privacy}","received_events_url":"https://api.github.com/users/rahulpalamuttam/received_events","type":"User","site_admin":false},"created_at":"2016-07-05T21:53:09Z","updated_at":"2016-07-05T21:53:09Z","body":"Thank you @kwhitehall \r\nI was actually indicating we add the reshape functionality to both the AbstractTensor and SciTensor.\r\nSciTensor delegates to abstract tensor.\r\n\r\nI also want to point out that instead of having the individual shape parameters being passed to the function (through scala's varargs), we pass in a shape array. This is consistent with how numpy, matlab, and other array libraries do it. \r\n\r\nso the api will look like this :\r\n\r\nsciT(\"var\").reshape(Array(33, 31, 181, 286))\r\nand\r\nsciT(\"var\").reshape(\"reshapedVar\", Array(33,31,181,286))"}}
{"actor":{"display_login":"rahulpalamuttam"}}