{"action":"created","issue":{"url":"https://api.github.com/repos/vatlab/SOS/issues/748","repository_url":"https://api.github.com/repos/vatlab/SOS","labels_url":"https://api.github.com/repos/vatlab/SOS/issues/748/labels{/name}","comments_url":"https://api.github.com/repos/vatlab/SOS/issues/748/comments","events_url":"https://api.github.com/repos/vatlab/SOS/issues/748/events","html_url":"https://github.com/vatlab/SOS/issues/748","id":248151471,"number":748,"title":"Improve error message for external tasks","user":{"login":"gaow","id":917985,"avatar_url":"https://avatars1.githubusercontent.com/u/917985?v=4","gravatar_id":"","url":"https://api.github.com/users/gaow","html_url":"https://github.com/gaow","followers_url":"https://api.github.com/users/gaow/followers","following_url":"https://api.github.com/users/gaow/following{/other_user}","gists_url":"https://api.github.com/users/gaow/gists{/gist_id}","starred_url":"https://api.github.com/users/gaow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gaow/subscriptions","organizations_url":"https://api.github.com/users/gaow/orgs","repos_url":"https://api.github.com/users/gaow/repos","events_url":"https://api.github.com/users/gaow/events{/privacy}","received_events_url":"https://api.github.com/users/gaow/received_events","type":"User","site_admin":false},"labels":[{"id":325435539,"url":"https://api.github.com/repos/vatlab/SOS/labels/request","name":"request","color":"159818","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-08-05T02:16:02Z","updated_at":"2017-08-09T03:53:34Z","closed_at":null,"body":"I find the error prompt here is not ideal:\r\n\r\n```\r\nERROR: 1a3273c5b127f0cb34aa5e2ac62f9d58 failed with RuntimeError error Failed to execute script (ret=1).\r\nPlease use command\r\n\tRscript \\\r\n\t  --default-packages=datasets,methods,utils,stats,grDevices,graphics \\\r\n\t  /home/gaow/tmp/test/.sos/core_fl_0_16_e3e26db7.R\r\nunder \"/home/gaow/tmp/test\" to test it.\r\n```\r\n\r\nFor this example the computation is quite heavy. It takes up to an hour to reproduce the error. When executed as non-task the error message would dump on the screen, which is ugly but can give people some hints. \r\n\r\nHowever here apparently one can see the error message via `sos status 1a3273c5b127f0cb34aa5e2ac62f9d58 -v 4`. That opens the err file which has more information including the prompt above. That can save people from re-running. I'm wondering if it is possible to distinguish between task/non-task case and ask users to use `sos status` to check the job, instead of suggesting a rerun."},"comment":{"url":"https://api.github.com/repos/vatlab/SOS/issues/comments/321145555","html_url":"https://github.com/vatlab/SOS/issues/748#issuecomment-321145555","issue_url":"https://api.github.com/repos/vatlab/SOS/issues/748","id":321145555,"user":{"login":"gaow","id":917985,"avatar_url":"https://avatars1.githubusercontent.com/u/917985?v=4","gravatar_id":"","url":"https://api.github.com/users/gaow","html_url":"https://github.com/gaow","followers_url":"https://api.github.com/users/gaow/followers","following_url":"https://api.github.com/users/gaow/following{/other_user}","gists_url":"https://api.github.com/users/gaow/gists{/gist_id}","starred_url":"https://api.github.com/users/gaow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gaow/subscriptions","organizations_url":"https://api.github.com/users/gaow/orgs","repos_url":"https://api.github.com/users/gaow/repos","events_url":"https://api.github.com/users/gaow/events{/privacy}","received_events_url":"https://api.github.com/users/gaow/received_events","type":"User","site_admin":false},"created_at":"2017-08-09T03:53:34Z","updated_at":"2017-08-09T03:53:34Z","body":"Oh no, I'm cool either way (though may think of some format improvements without expanding the output too much). I was just copying an old example from above. What am saying is:\r\n\r\n## Old behavior\r\n\r\n1. Show `failed to run <command> <script>` and quit\r\n\r\n## Current behavior\r\n1. Show `sos status ID -v4` to  direct users for more details, and quit\r\n2. User follow the instruction, and display\r\n   a. Exact error prompt\r\n   b. again the instruction to run `sos status`\r\n\r\nMy point is that current behavior 1 is good. Also 2a is very informative because I can immediately see what's the problem. However if I cannot eyeball the bug, or want to use some example script to reproduce it, then I hope to have old behavior 1 as the next step to replace the current 2b. "}}
{"actor":{"display_login":"gaow"}}