{"action":"created","issue":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/10227","repository_url":"https://api.github.com/repos/chapel-lang/chapel","labels_url":"https://api.github.com/repos/chapel-lang/chapel/issues/10227/labels{/name}","comments_url":"https://api.github.com/repos/chapel-lang/chapel/issues/10227/comments","events_url":"https://api.github.com/repos/chapel-lang/chapel/issues/10227/events","html_url":"https://github.com/chapel-lang/chapel/issues/10227","id":339632148,"node_id":"MDU6SXNzdWUzMzk2MzIxNDg=","number":10227,"title":"What to do for dataParTasksPerLocale < 0","user":{"login":"ronawho","id":1588337,"node_id":"MDQ6VXNlcjE1ODgzMzc=","avatar_url":"https://avatars1.githubusercontent.com/u/1588337?v=4","gravatar_id":"","url":"https://api.github.com/users/ronawho","html_url":"https://github.com/ronawho","followers_url":"https://api.github.com/users/ronawho/followers","following_url":"https://api.github.com/users/ronawho/following{/other_user}","gists_url":"https://api.github.com/users/ronawho/gists{/gist_id}","starred_url":"https://api.github.com/users/ronawho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ronawho/subscriptions","organizations_url":"https://api.github.com/users/ronawho/orgs","repos_url":"https://api.github.com/users/ronawho/repos","events_url":"https://api.github.com/users/ronawho/events{/privacy}","received_events_url":"https://api.github.com/users/ronawho/received_events","type":"User","site_admin":false},"labels":[{"id":496591534,"node_id":"MDU6TGFiZWw0OTY1OTE1MzQ=","url":"https://api.github.com/repos/chapel-lang/chapel/labels/area:%20Modules","name":"area: Modules","color":"d4c5f9","default":false}],"state":"open","locked":false,"assignee":{"login":"ronawho","id":1588337,"node_id":"MDQ6VXNlcjE1ODgzMzc=","avatar_url":"https://avatars1.githubusercontent.com/u/1588337?v=4","gravatar_id":"","url":"https://api.github.com/users/ronawho","html_url":"https://github.com/ronawho","followers_url":"https://api.github.com/users/ronawho/followers","following_url":"https://api.github.com/users/ronawho/following{/other_user}","gists_url":"https://api.github.com/users/ronawho/gists{/gist_id}","starred_url":"https://api.github.com/users/ronawho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ronawho/subscriptions","organizations_url":"https://api.github.com/users/ronawho/orgs","repos_url":"https://api.github.com/users/ronawho/repos","events_url":"https://api.github.com/users/ronawho/events{/privacy}","received_events_url":"https://api.github.com/users/ronawho/received_events","type":"User","site_admin":false},"assignees":[{"login":"ronawho","id":1588337,"node_id":"MDQ6VXNlcjE1ODgzMzc=","avatar_url":"https://avatars1.githubusercontent.com/u/1588337?v=4","gravatar_id":"","url":"https://api.github.com/users/ronawho","html_url":"https://github.com/ronawho","followers_url":"https://api.github.com/users/ronawho/followers","following_url":"https://api.github.com/users/ronawho/following{/other_user}","gists_url":"https://api.github.com/users/ronawho/gists{/gist_id}","starred_url":"https://api.github.com/users/ronawho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ronawho/subscriptions","organizations_url":"https://api.github.com/users/ronawho/orgs","repos_url":"https://api.github.com/users/ronawho/repos","events_url":"https://api.github.com/users/ronawho/events{/privacy}","received_events_url":"https://api.github.com/users/ronawho/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2018-07-09T22:43:21Z","updated_at":"2018-07-09T23:04:34Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"Currently we use `dataParTasksPerLocale=0` as a sentinel to say \"the implementation can decide how many tasks to use\" and we `halt()` for `dataParTasksPerLocale < 0`: \r\n\r\nhttps://github.com/chapel-lang/chapel/blob/b5577e1b54fdabbba5811aa371df74727794bece/modules/internal/DefaultRectangular.chpl#L27\r\n\r\nhttps://github.com/chapel-lang/chapel/blob/b5577e1b54fdabbba5811aa371df74727794bece/modules/standard/DynamicIters.chpl#L703-L713\r\n\r\nhalting seems excessive to me, so I was wondering if we should consider changing this to `dataParTasksPerLocale <= 0` means \"implementation can decide\". However, we may just be papering over a bug in user code or something, so maybe we should issue a warning?"},"comment":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/comments/403646998","html_url":"https://github.com/chapel-lang/chapel/issues/10227#issuecomment-403646998","issue_url":"https://api.github.com/repos/chapel-lang/chapel/issues/10227","id":403646998,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMzY0Njk5OA==","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-07-09T23:04:34Z","updated_at":"2018-07-09T23:04:34Z","author_association":"MEMBER","body":"We should warn if the user explicitly sets `dataParTasksPerLocale` to a negative value but I would be happy if, having done so, we replace that negative value with `here.maxTaskPar` and proceed with execution."}}
{"actor":{"display_login":"gbtitus"}}