{"action":"closed","issue":{"url":"https://api.github.com/repos/vega/vega-dataflow/issues/14","repository_url":"https://api.github.com/repos/vega/vega-dataflow","labels_url":"https://api.github.com/repos/vega/vega-dataflow/issues/14/labels{/name}","comments_url":"https://api.github.com/repos/vega/vega-dataflow/issues/14/comments","events_url":"https://api.github.com/repos/vega/vega-dataflow/issues/14/events","html_url":"https://github.com/vega/vega-dataflow/issues/14","id":193843069,"number":14,"title":"Review signal update return logic","user":{"login":"jheer","id":202799,"avatar_url":"https://avatars.githubusercontent.com/u/202799?v=3","gravatar_id":"","url":"https://api.github.com/users/jheer","html_url":"https://github.com/jheer","followers_url":"https://api.github.com/users/jheer/followers","following_url":"https://api.github.com/users/jheer/following{/other_user}","gists_url":"https://api.github.com/users/jheer/gists{/gist_id}","starred_url":"https://api.github.com/users/jheer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jheer/subscriptions","organizations_url":"https://api.github.com/users/jheer/orgs","repos_url":"https://api.github.com/users/jheer/repos","events_url":"https://api.github.com/users/jheer/events{/privacy}","received_events_url":"https://api.github.com/users/jheer/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-12-06T18:04:47Z","updated_at":"2016-12-06T22:49:25Z","closed_at":"2016-12-06T22:49:25Z","body":"Here: https://github.com/vega/vega-dataflow/blob/master/src/dataflow/on.js#L71\r\n\r\nWe return `undefined` when we skip the target, which may lead to unnecessary pulse propagation due to unmatched values on the update signal. While not necessarily a major problem (just performance overhead), this can lead to seemingly inconsistent behavior that is difficult to interpret when debugging (see vega/vega-parser#5)."}}
{"actor":{"display_login":"jheer"}}