{"action":"created","issue":{"url":"https://api.github.com/repos/ClementPinard/FlowNetPytorch/issues/36","repository_url":"https://api.github.com/repos/ClementPinard/FlowNetPytorch","labels_url":"https://api.github.com/repos/ClementPinard/FlowNetPytorch/issues/36/labels{/name}","comments_url":"https://api.github.com/repos/ClementPinard/FlowNetPytorch/issues/36/comments","events_url":"https://api.github.com/repos/ClementPinard/FlowNetPytorch/issues/36/events","html_url":"https://github.com/ClementPinard/FlowNetPytorch/pull/36","id":328779003,"node_id":"MDExOlB1bGxSZXF1ZXN0MTkyMjE4OTMx","number":36,"title":"Important fix to EPE calc","user":{"login":"bkvie","id":8351075,"node_id":"MDQ6VXNlcjgzNTEwNzU=","avatar_url":"https://avatars0.githubusercontent.com/u/8351075?v=4","gravatar_id":"","url":"https://api.github.com/users/bkvie","html_url":"https://github.com/bkvie","followers_url":"https://api.github.com/users/bkvie/followers","following_url":"https://api.github.com/users/bkvie/following{/other_user}","gists_url":"https://api.github.com/users/bkvie/gists{/gist_id}","starred_url":"https://api.github.com/users/bkvie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bkvie/subscriptions","organizations_url":"https://api.github.com/users/bkvie/orgs","repos_url":"https://api.github.com/users/bkvie/repos","events_url":"https://api.github.com/users/bkvie/events{/privacy}","received_events_url":"https://api.github.com/users/bkvie/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-06-02T21:35:51Z","updated_at":"2018-06-03T10:33:37Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/ClementPinard/FlowNetPytorch/pulls/36","html_url":"https://github.com/ClementPinard/FlowNetPytorch/pull/36","diff_url":"https://github.com/ClementPinard/FlowNetPytorch/pull/36.diff","patch_url":"https://github.com/ClementPinard/FlowNetPytorch/pull/36.patch"},"body":"This will be the first of several pull request extending the excellent code base. Comparing the EPE calc with the official version provided by the Middleburry data set, it came to my attention that torch.norm(...,2,2) creates the required flow_map, torch.norm(...,2,1) does not. Hope we can get this out quickly :)"},"comment":{"url":"https://api.github.com/repos/ClementPinard/FlowNetPytorch/issues/comments/394152624","html_url":"https://github.com/ClementPinard/FlowNetPytorch/pull/36#issuecomment-394152624","issue_url":"https://api.github.com/repos/ClementPinard/FlowNetPytorch/issues/36","id":394152624,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NDE1MjYyNA==","user":{"login":"bkvie","id":8351075,"node_id":"MDQ6VXNlcjgzNTEwNzU=","avatar_url":"https://avatars0.githubusercontent.com/u/8351075?v=4","gravatar_id":"","url":"https://api.github.com/users/bkvie","html_url":"https://github.com/bkvie","followers_url":"https://api.github.com/users/bkvie/followers","following_url":"https://api.github.com/users/bkvie/following{/other_user}","gists_url":"https://api.github.com/users/bkvie/gists{/gist_id}","starred_url":"https://api.github.com/users/bkvie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bkvie/subscriptions","organizations_url":"https://api.github.com/users/bkvie/orgs","repos_url":"https://api.github.com/users/bkvie/repos","events_url":"https://api.github.com/users/bkvie/events{/privacy}","received_events_url":"https://api.github.com/users/bkvie/received_events","type":"User","site_admin":false},"created_at":"2018-06-03T10:33:37Z","updated_at":"2018-06-03T10:33:37Z","author_association":"NONE","body":"In [4]: target_flow.shape\r\nOut[4]: torch.Size([8, 2, 80, 112])\r\n\r\nIn [5]: input_flow.shape\r\nOut[5]: torch.Size([8, 2, 80, 112])\r\n\r\nIn [9]: torch.norm(maps,2,2).shape\r\nOut[9]: torch.Size([8, 2, 112])\r\n\r\nIn [10]: torch.norm(maps,2,1).shape\r\nOut[10]: torch.Size([8, 80, 112])\r\n\r\nSo torch.norm(maps,2,1) is the right dimension. This however fails when evaluation EPE on inference as we do not have batches. Let me update EPE for inference."}}
{"actor":{"display_login":"bkvie"}}