{"action":"created","issue":{"url":"https://api.github.com/repos/vhbb/cmssw/issues/145","labels_url":"https://api.github.com/repos/vhbb/cmssw/issues/145/labels{/name}","comments_url":"https://api.github.com/repos/vhbb/cmssw/issues/145/comments","events_url":"https://api.github.com/repos/vhbb/cmssw/issues/145/events","html_url":"https://github.com/vhbb/cmssw/issues/145","id":103684718,"number":145,"title":"re-check GenJet clustering for GenHFHadronMatcher","user":{"login":"jpata","id":69717,"avatar_url":"https://avatars.githubusercontent.com/u/69717?v=3","gravatar_id":"","url":"https://api.github.com/users/jpata","html_url":"https://github.com/jpata","followers_url":"https://api.github.com/users/jpata/followers","following_url":"https://api.github.com/users/jpata/following{/other_user}","gists_url":"https://api.github.com/users/jpata/gists{/gist_id}","starred_url":"https://api.github.com/users/jpata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpata/subscriptions","organizations_url":"https://api.github.com/users/jpata/orgs","repos_url":"https://api.github.com/users/jpata/repos","events_url":"https://api.github.com/users/jpata/events{/privacy}","received_events_url":"https://api.github.com/users/jpata/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/vhbb/cmssw/milestones/4","html_url":"https://github.com/vhbb/cmssw/milestones/V13","labels_url":"https://api.github.com/repos/vhbb/cmssw/milestones/4/labels","id":1178757,"number":4,"title":"V13","description":"Meant for first data vs MC comparison, hence should run on data too","creator":{"login":"arizzi","id":5022358,"avatar_url":"https://avatars.githubusercontent.com/u/5022358?v=3","gravatar_id":"","url":"https://api.github.com/users/arizzi","html_url":"https://github.com/arizzi","followers_url":"https://api.github.com/users/arizzi/followers","following_url":"https://api.github.com/users/arizzi/following{/other_user}","gists_url":"https://api.github.com/users/arizzi/gists{/gist_id}","starred_url":"https://api.github.com/users/arizzi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arizzi/subscriptions","organizations_url":"https://api.github.com/users/arizzi/orgs","repos_url":"https://api.github.com/users/arizzi/repos","events_url":"https://api.github.com/users/arizzi/events{/privacy}","received_events_url":"https://api.github.com/users/arizzi/received_events","type":"User","site_admin":false},"open_issues":8,"closed_issues":11,"state":"open","created_at":"2015-06-24T07:18:50Z","updated_at":"2015-08-28T11:51:11Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2015-08-28T09:33:40Z","updated_at":"2015-08-31T12:18:33Z","closed_at":null,"body":"making an issue so I know to solve GenJet stuff from #67 which is closed.\r\n \r\nfrom Andrea:\r\nthose are the open points in my view:\r\n\r\nwe should not remake the gen jets as they should be the same but I understand that for the hadron matching this could be actually needed? can you confirm?\r\nin output we should have a single gen jet having all needed features, namely:\r\nthe standard flavour coming from PAT\r\nthe hadron matched flavour\r\nthe genNu correction\r\nif hadron matcher needs to remake its own jets, then we have to back match those to the original collection and just set a had-flavour attribute\r\n\r\nWith spring15, the genjets still need to be re-clustered. As far as I know, the original authors (Johannes Hauk, @bartonaz) are taking a look since June.\r\n \r\nThis is the error we get halfway through the processing.\r\n~~~\r\nBegin processing the 171st record. Run 1, Event 2982953, LumiSection 29870 at 28-Aug-2015 11:20:24.060 CEST\r\n----- Begin Fatal Exception 28-Aug-2015 11:20:24 CEST-----------------------\r\nAn exception of category 'ProductNotFound' occurred while\r\n   [0] Processing run: 1 lumi: 29870 event: 2982953\r\n   [1] Running path 'endpath'\r\n   [2] Calling event method for module PoolOutputModule/'OUT'\r\n   [3] Calling produce method for unscheduled module GenHFHadronMatcher/'matchGenBHadron'\r\n   [4] Calling produce method for unscheduled module JetFlavourClustering/'genJetFlavourPlusLeptonInfos'\r\nException Message:\r\nInvalidID get by product ID: invalid ProductID supplied\r\n   Additional Info:\r\n      [a] If you wish to continue processing events after a ProductNotFound exception,\r\nadd \"SkipEvent = cms.untracked.vstring('ProductNotFound')\" to the \"options\" PSet in the configuration.\r\n\r\n----- End Fatal Exception -------------------------------------------------\r\n~~~"},"comment":{"url":"https://api.github.com/repos/vhbb/cmssw/issues/comments/136356397","html_url":"https://github.com/vhbb/cmssw/issues/145#issuecomment-136356397","issue_url":"https://api.github.com/repos/vhbb/cmssw/issues/145","id":136356397,"user":{"login":"jpata","id":69717,"avatar_url":"https://avatars.githubusercontent.com/u/69717?v=3","gravatar_id":"","url":"https://api.github.com/users/jpata","html_url":"https://github.com/jpata","followers_url":"https://api.github.com/users/jpata/followers","following_url":"https://api.github.com/users/jpata/following{/other_user}","gists_url":"https://api.github.com/users/jpata/gists{/gist_id}","starred_url":"https://api.github.com/users/jpata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpata/subscriptions","organizations_url":"https://api.github.com/users/jpata/orgs","repos_url":"https://api.github.com/users/jpata/repos","events_url":"https://api.github.com/users/jpata/events{/privacy}","received_events_url":"https://api.github.com/users/jpata/received_events","type":"User","site_admin":false},"created_at":"2015-08-31T12:18:33Z","updated_at":"2015-08-31T12:18:33Z","body":"@bartonaz ah OK, thanks for the clarification. If the re-clustered jets are identical, then this matching is only for our convenience not to store gen-jets twice and still keep the original miniAOD definition of genjets. In short I'd simply like to fix what will be the GenHFHadronMatcher recipe: re-clustered jets or not, but I suppose we are still waiting for feedback from the experts. "}}