{"action":"created","issue":{"url":"https://api.github.com/repos/ImageEngine/gaffer/issues/1434","labels_url":"https://api.github.com/repos/ImageEngine/gaffer/issues/1434/labels{/name}","comments_url":"https://api.github.com/repos/ImageEngine/gaffer/issues/1434/comments","events_url":"https://api.github.com/repos/ImageEngine/gaffer/issues/1434/events","html_url":"https://github.com/ImageEngine/gaffer/pull/1434","id":100552083,"number":1434,"title":"Replaced all InputGenerators with ArrayPlugs","user":{"login":"johnhaddon","id":1133871,"avatar_url":"https://avatars.githubusercontent.com/u/1133871?v=3","gravatar_id":"","url":"https://api.github.com/users/johnhaddon","html_url":"https://github.com/johnhaddon","followers_url":"https://api.github.com/users/johnhaddon/followers","following_url":"https://api.github.com/users/johnhaddon/following{/other_user}","gists_url":"https://api.github.com/users/johnhaddon/gists{/gist_id}","starred_url":"https://api.github.com/users/johnhaddon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnhaddon/subscriptions","organizations_url":"https://api.github.com/users/johnhaddon/orgs","repos_url":"https://api.github.com/users/johnhaddon/repos","events_url":"https://api.github.com/users/johnhaddon/events{/privacy}","received_events_url":"https://api.github.com/users/johnhaddon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-08-12T14:07:45Z","updated_at":"2015-08-14T22:11:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ImageEngine/gaffer/pulls/1434","html_url":"https://github.com/ImageEngine/gaffer/pull/1434","diff_url":"https://github.com/ImageEngine/gaffer/pull/1434.diff","patch_url":"https://github.com/ImageEngine/gaffer/pull/1434.patch"},"body":"This is the full implementation of #1428 (ImageProcessors with ArrayPlug inputs), extended to allow ArrayPlug inputs for SceneProcessors, FilterProcessors and Switches. This allows us to remove the InputGenerator and FilterMixinBase classes entirely."},"comment":{"url":"https://api.github.com/repos/ImageEngine/gaffer/issues/comments/131252534","html_url":"https://github.com/ImageEngine/gaffer/pull/1434#issuecomment-131252534","issue_url":"https://api.github.com/repos/ImageEngine/gaffer/issues/1434","id":131252534,"user":{"login":"andrewkaufman","id":3979682,"avatar_url":"https://avatars.githubusercontent.com/u/3979682?v=3","gravatar_id":"","url":"https://api.github.com/users/andrewkaufman","html_url":"https://github.com/andrewkaufman","followers_url":"https://api.github.com/users/andrewkaufman/followers","following_url":"https://api.github.com/users/andrewkaufman/following{/other_user}","gists_url":"https://api.github.com/users/andrewkaufman/gists{/gist_id}","starred_url":"https://api.github.com/users/andrewkaufman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrewkaufman/subscriptions","organizations_url":"https://api.github.com/users/andrewkaufman/orgs","repos_url":"https://api.github.com/users/andrewkaufman/repos","events_url":"https://api.github.com/users/andrewkaufman/events{/privacy}","received_events_url":"https://api.github.com/users/andrewkaufman/received_events","type":"User","site_admin":false},"created_at":"2015-08-14T22:11:48Z","updated_at":"2015-08-14T22:11:48Z","body":"Testing this on my image processing scene, I had a broken expression on startup, because I was accessing the inPlug of the ImageSwitch in the expression that drives it's index.\r\n\r\nerror:\r\n`ERROR : Expression::plugSet : Plug \"AlphaSwitch.in\" does not exist`\r\n\r\noriginal expression text:\r\n`parent['AlphaSwitch']['index'] = \"A\" not in parent['AlphaSwitch']['in'][\"channelNames\"]`\r\n\r\nthis update didn't work, though I thought it should have:\r\n`parent['AlphaSwitch']['index'] = \"A\" not in parent['AlphaSwitch']['in'][0][\"channelNames\"]`\r\n\r\nthis one worked:\r\n`parent['AlphaSwitch']['index'] = \"A\" not in parent['AlphaSwitch']['in']['in0'][\"channelNames\"]`\r\n\r\nAnyways, this seems like enough of an edge case not to worry about compatibility for, unless you think its super simple. I doubt any users were writing expressions that access the input plugs like that...\r\n"}}