{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/8808","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/8808/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/8808/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/8808/events","html_url":"https://github.com/cms-sw/cmssw/pull/8808","id":69781096,"number":8808,"title":"allowing E/g HLT to be seeded by L1 jets (74X edition)","user":{"login":"Sam-Harper","id":5021403,"avatar_url":"https://avatars.githubusercontent.com/u/5021403?v=3","gravatar_id":"","url":"https://api.github.com/users/Sam-Harper","html_url":"https://github.com/Sam-Harper","followers_url":"https://api.github.com/users/Sam-Harper/followers","following_url":"https://api.github.com/users/Sam-Harper/following{/other_user}","gists_url":"https://api.github.com/users/Sam-Harper/gists{/gist_id}","starred_url":"https://api.github.com/users/Sam-Harper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sam-Harper/subscriptions","organizations_url":"https://api.github.com/users/Sam-Harper/orgs","repos_url":"https://api.github.com/users/Sam-Harper/repos","events_url":"https://api.github.com/users/Sam-Harper/events{/privacy}","received_events_url":"https://api.github.com/users/Sam-Harper/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-available","name":"comparison-available","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/fully-signed","name":"fully-signed","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/hlt-approved","name":"hlt-approved","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-approved","name":"tests-approved","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/50","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_7_4_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/50/labels","id":875803,"number":50,"title":"Next CMSSW_7_4_X","description":"","creator":{"login":"nclopezo","id":1944922,"avatar_url":"https://avatars.githubusercontent.com/u/1944922?v=3","gravatar_id":"","url":"https://api.github.com/users/nclopezo","html_url":"https://github.com/nclopezo","followers_url":"https://api.github.com/users/nclopezo/followers","following_url":"https://api.github.com/users/nclopezo/following{/other_user}","gists_url":"https://api.github.com/users/nclopezo/gists{/gist_id}","starred_url":"https://api.github.com/users/nclopezo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nclopezo/subscriptions","organizations_url":"https://api.github.com/users/nclopezo/orgs","repos_url":"https://api.github.com/users/nclopezo/repos","events_url":"https://api.github.com/users/nclopezo/events{/privacy}","received_events_url":"https://api.github.com/users/nclopezo/received_events","type":"User","site_admin":false},"open_issues":31,"closed_issues":1039,"state":"open","created_at":"2014-11-21T15:23:58Z","updated_at":"2015-04-27T09:16:23Z","due_on":"2015-01-26T08:00:00Z","closed_at":null},"comments":12,"created_at":"2015-04-21T08:23:54Z","updated_at":"2015-04-28T10:00:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/8808","html_url":"https://github.com/cms-sw/cmssw/pull/8808","diff_url":"https://github.com/cms-sw/cmssw/pull/8808.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/8808.patch"},"body":"This is the 74X edition of a change to the E/g HLT to allow it to be seeded by L1 jets. It is discussed in  https://github.com/cms-sw/cmssw/pull/8804."},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/96997503","html_url":"https://github.com/cms-sw/cmssw/pull/8808#issuecomment-96997503","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/8808","id":96997503,"user":{"login":"Sam-Harper","id":5021403,"avatar_url":"https://avatars.githubusercontent.com/u/5021403?v=3","gravatar_id":"","url":"https://api.github.com/users/Sam-Harper","html_url":"https://github.com/Sam-Harper","followers_url":"https://api.github.com/users/Sam-Harper/followers","following_url":"https://api.github.com/users/Sam-Harper/following{/other_user}","gists_url":"https://api.github.com/users/Sam-Harper/gists{/gist_id}","starred_url":"https://api.github.com/users/Sam-Harper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sam-Harper/subscriptions","organizations_url":"https://api.github.com/users/Sam-Harper/orgs","repos_url":"https://api.github.com/users/Sam-Harper/repos","events_url":"https://api.github.com/users/Sam-Harper/events{/privacy}","received_events_url":"https://api.github.com/users/Sam-Harper/received_events","type":"User","site_admin":false},"created_at":"2015-04-28T10:00:45Z","updated_at":"2015-04-28T10:00:45Z","body":"So yes you are strictly correct, for all possible configs, this does not reproduce the old behavior. However for any sensible HLT config it will do. If the L1GtSeed is not a Jet, looking for the jets will have zero effect as they are not there. Only if you're requested a Jet seed in your L1 seeding module will the jet collection be filled. And no E/gamma path did this because it would fail at this matching step. Now it will pass. The idea was that this module automatically accepts whatever seeds was specified by the L1 seeding module. Otherwise you have to enable it both in this module (ie allow jets) and the seed module (seed with a jet module). I'll double check i've understood the logic correctly though. "}}