{"action":"created","issue":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/11041","repository_url":"https://api.github.com/repos/chapel-lang/chapel","labels_url":"https://api.github.com/repos/chapel-lang/chapel/issues/11041/labels{/name}","comments_url":"https://api.github.com/repos/chapel-lang/chapel/issues/11041/comments","events_url":"https://api.github.com/repos/chapel-lang/chapel/issues/11041/events","html_url":"https://github.com/chapel-lang/chapel/pull/11041","id":358162607,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE0MDAyMTE4","number":11041,"title":"Report message size under verbose communication","user":{"login":"e-kayrakli","id":4141670,"node_id":"MDQ6VXNlcjQxNDE2NzA=","avatar_url":"https://avatars3.githubusercontent.com/u/4141670?v=4","gravatar_id":"","url":"https://api.github.com/users/e-kayrakli","html_url":"https://github.com/e-kayrakli","followers_url":"https://api.github.com/users/e-kayrakli/followers","following_url":"https://api.github.com/users/e-kayrakli/following{/other_user}","gists_url":"https://api.github.com/users/e-kayrakli/gists{/gist_id}","starred_url":"https://api.github.com/users/e-kayrakli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/e-kayrakli/subscriptions","organizations_url":"https://api.github.com/users/e-kayrakli/orgs","repos_url":"https://api.github.com/users/e-kayrakli/repos","events_url":"https://api.github.com/users/e-kayrakli/events{/privacy}","received_events_url":"https://api.github.com/users/e-kayrakli/received_events","type":"User","site_admin":false},"labels":[{"id":576489172,"node_id":"MDU6TGFiZWw1NzY0ODkxNzI=","url":"https://api.github.com/repos/chapel-lang/chapel/labels/post-release","name":"post-release","color":"ff944d","default":false},{"id":195153428,"node_id":"MDU6TGFiZWwxOTUxNTM0Mjg=","url":"https://api.github.com/repos/chapel-lang/chapel/labels/stat:%20Work%20in%20Progress","name":"stat: Work in Progress","color":"c5def5","default":false}],"state":"open","locked":false,"assignee":{"login":"gbtitus","id":7740880,"node_id":"MDQ6VXNlcjc3NDA4ODA=","avatar_url":"https://avatars3.githubusercontent.com/u/7740880?v=4","gravatar_id":"","url":"https://api.github.com/users/gbtitus","html_url":"https://github.com/gbtitus","followers_url":"https://api.github.com/users/gbtitus/followers","following_url":"https://api.github.com/users/gbtitus/following{/other_user}","gists_url":"https://api.github.com/users/gbtitus/gists{/gist_id}","starred_url":"https://api.github.com/users/gbtitus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gbtitus/subscriptions","organizations_url":"https://api.github.com/users/gbtitus/orgs","repos_url":"https://api.github.com/users/gbtitus/repos","events_url":"https://api.github.com/users/gbtitus/events{/privacy}","received_events_url":"https://api.github.com/users/gbtitus/received_events","type":"User","site_admin":false},"assignees":[{"login":"gbtitus","id":7740880,"node_id":"MDQ6VXNlcjc3NDA4ODA=","avatar_url":"https://avatars3.githubusercontent.com/u/7740880?v=4","gravatar_id":"","url":"https://api.github.com/users/gbtitus","html_url":"https://github.com/gbtitus","followers_url":"https://api.github.com/users/gbtitus/followers","following_url":"https://api.github.com/users/gbtitus/following{/other_user}","gists_url":"https://api.github.com/users/gbtitus/gists{/gist_id}","starred_url":"https://api.github.com/users/gbtitus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gbtitus/subscriptions","organizations_url":"https://api.github.com/users/gbtitus/orgs","repos_url":"https://api.github.com/users/gbtitus/repos","events_url":"https://api.github.com/users/gbtitus/events{/privacy}","received_events_url":"https://api.github.com/users/gbtitus/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2018-09-07T18:11:56Z","updated_at":"2018-09-18T20:56:39Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/chapel-lang/chapel/pulls/11041","html_url":"https://github.com/chapel-lang/chapel/pull/11041","diff_url":"https://github.com/chapel-lang/chapel/pull/11041.diff","patch_url":"https://github.com/chapel-lang/chapel/pull/11041.patch"},"body":"This PR adds message size to the verbose communication output. The message size is appended to the current verbose comm output as `, N bytes`\r\n\r\n~Currently work-in-progress. TODO:~\r\n\r\n- [x] Add message size for ugni\r\n- [x] Confirm we don't need to modify anything for ofi\r\n- [x] Confirm `%zu` is portable enough for printing `size_t` variables\r\n- [x] Any test changes?\r\n"},"comment":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/comments/422551640","html_url":"https://github.com/chapel-lang/chapel/pull/11041#issuecomment-422551640","issue_url":"https://api.github.com/repos/chapel-lang/chapel/issues/11041","id":422551640,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMjU1MTY0MA==","user":{"login":"gbtitus","id":7740880,"node_id":"MDQ6VXNlcjc3NDA4ODA=","avatar_url":"https://avatars3.githubusercontent.com/u/7740880?v=4","gravatar_id":"","url":"https://api.github.com/users/gbtitus","html_url":"https://github.com/gbtitus","followers_url":"https://api.github.com/users/gbtitus/followers","following_url":"https://api.github.com/users/gbtitus/following{/other_user}","gists_url":"https://api.github.com/users/gbtitus/gists{/gist_id}","starred_url":"https://api.github.com/users/gbtitus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gbtitus/subscriptions","organizations_url":"https://api.github.com/users/gbtitus/orgs","repos_url":"https://api.github.com/users/gbtitus/repos","events_url":"https://api.github.com/users/gbtitus/events{/privacy}","received_events_url":"https://api.github.com/users/gbtitus/received_events","type":"User","site_admin":false},"created_at":"2018-09-18T20:56:39Z","updated_at":"2018-09-18T20:56:39Z","author_association":"MEMBER","body":"This looks great but sure enough, we do not test this feature!  Would you mind adding your little feature test shown above, say in a new `test/multilocale/e-kayrakli/needMultiLocales/` directory?  It could look a lot like what's in `test/multilocale/gbt/needMultiLocales/`, but I think the only files you'll need are:\r\n```\r\nverboseComm.chpl\r\nverboseComm.good\r\nNUMLOCALES\r\nSKIPIF\r\n```"}}
{"actor":{"display_login":"gbtitus"}}