{"action":"created","issue":{"url":"https://api.github.com/repos/UCATLAS/xAODAnaHelpers/issues/521","repository_url":"https://api.github.com/repos/UCATLAS/xAODAnaHelpers","labels_url":"https://api.github.com/repos/UCATLAS/xAODAnaHelpers/issues/521/labels{/name}","comments_url":"https://api.github.com/repos/UCATLAS/xAODAnaHelpers/issues/521/comments","events_url":"https://api.github.com/repos/UCATLAS/xAODAnaHelpers/issues/521/events","html_url":"https://github.com/UCATLAS/xAODAnaHelpers/pull/521","id":137141098,"number":521,"title":"reordering makeSubsetCont to avoid crash","user":{"login":"JamesSaxon","id":12751805,"avatar_url":"https://avatars.githubusercontent.com/u/12751805?v=3","gravatar_id":"","url":"https://api.github.com/users/JamesSaxon","html_url":"https://github.com/JamesSaxon","followers_url":"https://api.github.com/users/JamesSaxon/followers","following_url":"https://api.github.com/users/JamesSaxon/following{/other_user}","gists_url":"https://api.github.com/users/JamesSaxon/gists{/gist_id}","starred_url":"https://api.github.com/users/JamesSaxon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JamesSaxon/subscriptions","organizations_url":"https://api.github.com/users/JamesSaxon/orgs","repos_url":"https://api.github.com/users/JamesSaxon/repos","events_url":"https://api.github.com/users/JamesSaxon/events{/privacy}","received_events_url":"https://api.github.com/users/JamesSaxon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2016-02-29T03:34:59Z","updated_at":"2016-03-01T02:01:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/UCATLAS/xAODAnaHelpers/pulls/521","html_url":"https://github.com/UCATLAS/xAODAnaHelpers/pull/521","diff_url":"https://github.com/UCATLAS/xAODAnaHelpers/pull/521.diff","patch_url":"https://github.com/UCATLAS/xAODAnaHelpers/pull/521.patch"},"body":"This regards the ```makeSubsetCont()``` method.\r\n\r\nIn earlier versions, like 03-22, the const accessor was created for every item, but after the \"continue\" for CALIBRATOR: \r\n\r\n```static SG::AuxElement::ConstAccessor<char> myAccessor(flagSelect);```\r\n\r\nI get the following error for some derivations, when running ElectronCalibrator: \r\n\r\n```Exception: void EL::Driver::submit(const EL::Job& job, const string& location) =>\r\n    SG::ExcAuxTypeMismatch: Type mismatch for aux variable `::' (525); old type is float new type is char (C++ exception)```\r\n\r\nIn that case, ```makeSubsetCont()``` is called with ```flagSelect``` empty.  If I sandwich the accessor creator -- now without static -- with error statements, I see that the accessor is indeed the issue..\r\n\r\nThe issue can be avoided by making a separate loop, and only creating the accessor for subset containers that are not full copies."},"comment":{"url":"https://api.github.com/repos/UCATLAS/xAODAnaHelpers/issues/comments/190493820","html_url":"https://github.com/UCATLAS/xAODAnaHelpers/pull/521#issuecomment-190493820","issue_url":"https://api.github.com/repos/UCATLAS/xAODAnaHelpers/issues/521","id":190493820,"user":{"login":"xAH-bot","id":13026597,"avatar_url":"https://avatars.githubusercontent.com/u/13026597?v=3","gravatar_id":"","url":"https://api.github.com/users/xAH-bot","html_url":"https://github.com/xAH-bot","followers_url":"https://api.github.com/users/xAH-bot/followers","following_url":"https://api.github.com/users/xAH-bot/following{/other_user}","gists_url":"https://api.github.com/users/xAH-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/xAH-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xAH-bot/subscriptions","organizations_url":"https://api.github.com/users/xAH-bot/orgs","repos_url":"https://api.github.com/users/xAH-bot/repos","events_url":"https://api.github.com/users/xAH-bot/events{/privacy}","received_events_url":"https://api.github.com/users/xAH-bot/received_events","type":"User","site_admin":false},"created_at":"2016-03-01T02:01:56Z","updated_at":"2016-03-01T02:01:56Z","body":"Everything checked out ok, :+1: to merge. :beers:\n\nCommit sha1: 4d4a28f63c2c494f866c1926cd0090134f1ac3ff\n"}}