{"action":"created","issue":{"url":"https://api.github.com/repos/scijava/scijava-common/issues/246","repository_url":"https://api.github.com/repos/scijava/scijava-common","labels_url":"https://api.github.com/repos/scijava/scijava-common/issues/246/labels{/name}","comments_url":"https://api.github.com/repos/scijava/scijava-common/issues/246/comments","events_url":"https://api.github.com/repos/scijava/scijava-common/issues/246/events","html_url":"https://github.com/scijava/scijava-common/issues/246","id":176186248,"number":246,"title":"Users should be warned of missing script output","user":{"login":"hinerm","id":1123809,"avatar_url":"https://avatars.githubusercontent.com/u/1123809?v=3","gravatar_id":"","url":"https://api.github.com/users/hinerm","html_url":"https://github.com/hinerm","followers_url":"https://api.github.com/users/hinerm/followers","following_url":"https://api.github.com/users/hinerm/following{/other_user}","gists_url":"https://api.github.com/users/hinerm/gists{/gist_id}","starred_url":"https://api.github.com/users/hinerm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hinerm/subscriptions","organizations_url":"https://api.github.com/users/hinerm/orgs","repos_url":"https://api.github.com/users/hinerm/repos","events_url":"https://api.github.com/users/hinerm/events{/privacy}","received_events_url":"https://api.github.com/users/hinerm/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-09-10T15:48:58Z","updated_at":"2016-09-12T13:45:10Z","closed_at":null,"body":"Consider this modified `greeting.py` script:\r\n\r\n```\r\n# @String(label=\"Please enter your name\",description=\"Name field\") name\r\n# @OUTPUT String greeting_missing\r\n\r\ngreeting = \"Hello, \" + name + \"!\"\r\n```\r\n\r\nWhen I run this from the script editor there's no apparent error message, log output or warning. The script just silently does nothing.\r\n\r\nThis could be confusing, especially in more complicated scripts."},"comment":{"url":"https://api.github.com/repos/scijava/scijava-common/issues/comments/246351455","html_url":"https://github.com/scijava/scijava-common/issues/246#issuecomment-246351455","issue_url":"https://api.github.com/repos/scijava/scijava-common/issues/246","id":246351455,"user":{"login":"ctrueden","id":556626,"avatar_url":"https://avatars.githubusercontent.com/u/556626?v=3","gravatar_id":"","url":"https://api.github.com/users/ctrueden","html_url":"https://github.com/ctrueden","followers_url":"https://api.github.com/users/ctrueden/followers","following_url":"https://api.github.com/users/ctrueden/following{/other_user}","gists_url":"https://api.github.com/users/ctrueden/gists{/gist_id}","starred_url":"https://api.github.com/users/ctrueden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ctrueden/subscriptions","organizations_url":"https://api.github.com/users/ctrueden/orgs","repos_url":"https://api.github.com/users/ctrueden/repos","events_url":"https://api.github.com/users/ctrueden/events{/privacy}","received_events_url":"https://api.github.com/users/ctrueden/received_events","type":"User","site_admin":false},"created_at":"2016-09-12T13:45:10Z","updated_at":"2016-09-12T13:45:10Z","body":"Unfortunately, I do not believe JSR-223 can, in general, differentiate\nbetween null and missing.\n\nOn Sep 12, 2016 8:00 AM, \"Mark Hiner\" <notifications@github.com> wrote:\n\n> Well, if a script has declared @OUTPUT values isn't there kind of an\n> implicit contract that you are asking the framework to do something with\n> those values? Especially in the case of something like a Dataset.\n>\n> One possibility might be to punt to the output handler, where you look up\n> the handler by output type and let them respond based on the actual value.\n>\n> Also, the intent of this issue (which I think was not completely clear\n> from my initial writeup) was that we may want to differentiate between\n> null values and missing values (e.g. I have an @OUTPUT referencing a\n> variable name that is undefined in my scripting environment). The latter\n> case seems less ambiguous that something has gone wrong in\n> programming/program state.\n>\n> —\n> You are receiving this because you commented.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/scijava/scijava-common/issues/246#issuecomment-246339392>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/AAh-UgslPOiMGKr-K_a4TbO3jdo-7zX2ks5qpUzMgaJpZM4J5wEe>\n> .\n>\n"}}
{"actor":{"display_login":"ctrueden"}}