{"action":"created","issue":{"url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/365","repository_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn","labels_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/365/labels{/name}","comments_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/365/comments","events_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/365/events","html_url":"https://github.com/tiny-dnn/tiny-dnn/issues/365","id":185894155,"number":365,"title":"Fail to integrate tiny-dnn with NNPack","user":{"login":"azsane","id":9714904,"avatar_url":"https://avatars.githubusercontent.com/u/9714904?v=3","gravatar_id":"","url":"https://api.github.com/users/azsane","html_url":"https://github.com/azsane","followers_url":"https://api.github.com/users/azsane/followers","following_url":"https://api.github.com/users/azsane/following{/other_user}","gists_url":"https://api.github.com/users/azsane/gists{/gist_id}","starred_url":"https://api.github.com/users/azsane/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/azsane/subscriptions","organizations_url":"https://api.github.com/users/azsane/orgs","repos_url":"https://api.github.com/users/azsane/repos","events_url":"https://api.github.com/users/azsane/events{/privacy}","received_events_url":"https://api.github.com/users/azsane/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-10-28T10:50:37Z","updated_at":"2016-12-01T08:30:14Z","closed_at":null,"body":"Hello,I'm trying to use NNPack to improve conv layer performance,but encounted serveral problems.  \n- NNPack API change (conv2d_op_nnpack.h)\n  - In conv2d_op_nnpack.h _nnp_convolution_kernel_transform_strategy_  was not found .Due to API it may be nnp_convolution_transform_strategy\n  - nnp_convolution_inference add a stride parameter as output_sampling but in function this parameter is omitted\n  -  nnp_convolution_algorithm_auto will cause kernel 7 pad 3 stride 2 to throw a algorithm not supported exception\n- convolution_layer \n  - this layer don't use engine for backend nnp  ,need to set it manually \n  - in forward ,function always pad the input,but nnp_convolution_inference would perform padding.And strangely,function will pass padded input to nnp ,so padding twice?\n\nAfter change some code to solve problems above.I run into a segment fault .\nperhaps caused by   code in _conv2d_op_nnpack_\n\n```\nfloat* output_ptr = reinterpret_cast<float*>(&out_data[0]);\n```\n\nIs this caused by NNPack version diff?or should I do some processing?  \n"},"comment":{"url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/comments/264109531","html_url":"https://github.com/tiny-dnn/tiny-dnn/issues/365#issuecomment-264109531","issue_url":"https://api.github.com/repos/tiny-dnn/tiny-dnn/issues/365","id":264109531,"user":{"login":"bhack","id":1710528,"avatar_url":"https://avatars.githubusercontent.com/u/1710528?v=3","gravatar_id":"","url":"https://api.github.com/users/bhack","html_url":"https://github.com/bhack","followers_url":"https://api.github.com/users/bhack/followers","following_url":"https://api.github.com/users/bhack/following{/other_user}","gists_url":"https://api.github.com/users/bhack/gists{/gist_id}","starred_url":"https://api.github.com/users/bhack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bhack/subscriptions","organizations_url":"https://api.github.com/users/bhack/orgs","repos_url":"https://api.github.com/users/bhack/repos","events_url":"https://api.github.com/users/bhack/events{/privacy}","received_events_url":"https://api.github.com/users/bhack/received_events","type":"User","site_admin":false},"created_at":"2016-12-01T08:30:14Z","updated_at":"2016-12-01T08:30:14Z","body":"What is the status of this?"}}
{"actor":{"display_login":"bhack"}}