{"action":"created","issue":{"url":"https://api.github.com/repos/HenrikBengtsson/future/issues/25","repository_url":"https://api.github.com/repos/HenrikBengtsson/future","labels_url":"https://api.github.com/repos/HenrikBengtsson/future/issues/25/labels{/name}","comments_url":"https://api.github.com/repos/HenrikBengtsson/future/issues/25/comments","events_url":"https://api.github.com/repos/HenrikBengtsson/future/issues/25/events","html_url":"https://github.com/HenrikBengtsson/future/issues/25","id":121513008,"number":25,"title":"Return warnings for futures with plan multicore","user":{"login":"drknexus","id":1847158,"avatar_url":"https://avatars.githubusercontent.com/u/1847158?v=3","gravatar_id":"","url":"https://api.github.com/users/drknexus","html_url":"https://github.com/drknexus","followers_url":"https://api.github.com/users/drknexus/followers","following_url":"https://api.github.com/users/drknexus/following{/other_user}","gists_url":"https://api.github.com/users/drknexus/gists{/gist_id}","starred_url":"https://api.github.com/users/drknexus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drknexus/subscriptions","organizations_url":"https://api.github.com/users/drknexus/orgs","repos_url":"https://api.github.com/users/drknexus/repos","events_url":"https://api.github.com/users/drknexus/events{/privacy}","received_events_url":"https://api.github.com/users/drknexus/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/HenrikBengtsson/future/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/HenrikBengtsson/future/labels/feature%20request","name":"feature request","color":"c5def5"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2015-12-10T15:36:44Z","updated_at":"2016-07-14T12:09:18Z","closed_at":null,"body":"Consider:\r\n```\r\nv %<=% {Sys.sleep(3);warning(\"Warning\");10} %plan% lazy\r\nv\r\nv %<=% {Sys.sleep(3);warning(\"No warning?\");10} %plan% multicore\r\nv\r\n```\r\n\r\nUnlike errors, warnings are not passed forward under the multicore plan.\r\n\r\nAlso, amazing package.  I wish I'd discovered it sooner."},"comment":{"url":"https://api.github.com/repos/HenrikBengtsson/future/issues/comments/232647567","html_url":"https://github.com/HenrikBengtsson/future/issues/25#issuecomment-232647567","issue_url":"https://api.github.com/repos/HenrikBengtsson/future/issues/25","id":232647567,"user":{"login":"drknexus","id":1847158,"avatar_url":"https://avatars.githubusercontent.com/u/1847158?v=3","gravatar_id":"","url":"https://api.github.com/users/drknexus","html_url":"https://github.com/drknexus","followers_url":"https://api.github.com/users/drknexus/followers","following_url":"https://api.github.com/users/drknexus/following{/other_user}","gists_url":"https://api.github.com/users/drknexus/gists{/gist_id}","starred_url":"https://api.github.com/users/drknexus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drknexus/subscriptions","organizations_url":"https://api.github.com/users/drknexus/orgs","repos_url":"https://api.github.com/users/drknexus/repos","events_url":"https://api.github.com/users/drknexus/events{/privacy}","received_events_url":"https://api.github.com/users/drknexus/received_events","type":"User","site_admin":false},"created_at":"2016-07-14T12:09:18Z","updated_at":"2016-07-14T12:09:18Z","body":"I think `options(warn=1L)` will provide the behavior I was expecting for multicore, i.e. roughly consistent with eager futures.\r\n\r\nAs for what is optimal... that's as you say a bit more murky.  On one hand, one wants to immediately know when the code in the future isn't behaving quite as expected.  On the other hand, the warning is inherently relevant to the value that is being returned from the future.\r\n\r\nBetween the two options, for my part, I lean towards a preference of re-signaling the error/messages as soon as value(f) is called."}}
{"actor":{"display_login":"drknexus"}}