{"action":"created","issue":{"url":"https://api.github.com/repos/losalamos/Draco/issues/103","repository_url":"https://api.github.com/repos/losalamos/Draco","labels_url":"https://api.github.com/repos/losalamos/Draco/issues/103/labels{/name}","comments_url":"https://api.github.com/repos/losalamos/Draco/issues/103/comments","events_url":"https://api.github.com/repos/losalamos/Draco/issues/103/events","html_url":"https://github.com/losalamos/Draco/issues/103","id":187992054,"number":103,"title":"Invert_Comm_Map should communicate data size to be exchanged","user":{"login":"lowrie","id":4357034,"avatar_url":"https://avatars.githubusercontent.com/u/4357034?v=3","gravatar_id":"","url":"https://api.github.com/users/lowrie","html_url":"https://github.com/lowrie","followers_url":"https://api.github.com/users/lowrie/followers","following_url":"https://api.github.com/users/lowrie/following{/other_user}","gists_url":"https://api.github.com/users/lowrie/gists{/gist_id}","starred_url":"https://api.github.com/users/lowrie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lowrie/subscriptions","organizations_url":"https://api.github.com/users/lowrie/orgs","repos_url":"https://api.github.com/users/lowrie/repos","events_url":"https://api.github.com/users/lowrie/events{/privacy}","received_events_url":"https://api.github.com/users/lowrie/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-11-08T13:30:36Z","updated_at":"2016-11-08T16:54:13Z","closed_at":null,"body":"c4's Invert_Comm_Map currently takes a std::vector<int> as input and output arguments, which are the vector of procs that the current proc will send/receive information to/from.  Instead, it should take a std::map<int, size_t>, that maps proc number to the data size to be sent/received.  Here, \"size\" is up to the user; it could be the number of elements in a container, or the actual number of bytes; it doesn't matter to this routine.\r\n\r\nThis change should give a very small decrease in performance, because size_t values would be communicated, instead of int values.  But it would allow users to eliminate the extra communication step of determining the size of information to be received."},"comment":{"url":"https://api.github.com/repos/losalamos/Draco/issues/comments/259192883","html_url":"https://github.com/losalamos/Draco/issues/103#issuecomment-259192883","issue_url":"https://api.github.com/repos/losalamos/Draco/issues/103","id":259192883,"user":{"login":"alexrlongne","id":9088671,"avatar_url":"https://avatars.githubusercontent.com/u/9088671?v=3","gravatar_id":"","url":"https://api.github.com/users/alexrlongne","html_url":"https://github.com/alexrlongne","followers_url":"https://api.github.com/users/alexrlongne/followers","following_url":"https://api.github.com/users/alexrlongne/following{/other_user}","gists_url":"https://api.github.com/users/alexrlongne/gists{/gist_id}","starred_url":"https://api.github.com/users/alexrlongne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexrlongne/subscriptions","organizations_url":"https://api.github.com/users/alexrlongne/orgs","repos_url":"https://api.github.com/users/alexrlongne/repos","events_url":"https://api.github.com/users/alexrlongne/events{/privacy}","received_events_url":"https://api.github.com/users/alexrlongne/received_events","type":"User","site_admin":false},"created_at":"2016-11-08T16:54:13Z","updated_at":"2016-11-08T16:54:13Z","body":"I agree. Then instead of writing a \"1\" with RMA operations for a flag in the invert comm map routine you can just write the size (or number of elements) that should be received from that rank."}}
{"actor":{"display_login":"alexrlongne"}}