{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/7659","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7659/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7659/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7659/events","html_url":"https://github.com/cms-sw/cmssw/pull/7659","id":57235542,"number":7659,"title":"Mini aod substructure pieces overhaul","user":{"login":"rappoccio","id":4267705,"avatar_url":"https://avatars.githubusercontent.com/u/4267705?v=3","gravatar_id":"","url":"https://api.github.com/users/rappoccio","html_url":"https://github.com/rappoccio","followers_url":"https://api.github.com/users/rappoccio/followers","following_url":"https://api.github.com/users/rappoccio/following{/other_user}","gists_url":"https://api.github.com/users/rappoccio/gists{/gist_id}","starred_url":"https://api.github.com/users/rappoccio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rappoccio/subscriptions","organizations_url":"https://api.github.com/users/rappoccio/orgs","repos_url":"https://api.github.com/users/rappoccio/repos","events_url":"https://api.github.com/users/rappoccio/events{/privacy}","received_events_url":"https://api.github.com/users/rappoccio/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/analysis-pending","name":"analysis-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fef2c0"},{"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/pending-signatures","name":"pending-signatures","color":"eb6420"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-pending","name":"tests-pending","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/50","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":40,"closed_issues":639,"state":"open","created_at":"2014-11-21T15:23:58Z","updated_at":"2015-02-11T15:03:05Z","due_on":"2015-01-26T08:00:00Z","closed_at":null},"comments":6,"created_at":"2015-02-10T20:58:04Z","updated_at":"2015-02-11T15:04:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/7659","html_url":"https://github.com/cms-sw/cmssw/pull/7659","diff_url":"https://github.com/cms-sw/cmssw/pull/7659.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/7659.patch"},"body":"Making new PR to accomplish the goals in  #7606, after code review. The only major change is to write out ONLY subjets that are associated to selected jets. \r\n\r\n====== EVENT Content =======\r\nThe AK8 jet content is therefore : \r\n\r\n- Add all AK8 jets for pt > 50 GeV\r\n   - If AK8 jet pt > 200:\r\n      - Add top-tagging and soft drop tagger info. \r\n      - Run b-tagging and JECs on subjets\r\n\r\nSo if the AK8 jet has pt between 50 and 200 GeV, the substructure algorithms are not run. If pt > 200 GeV, The subjets are written along with the \"fat\" jets in the BoostedJetMerger, and then are rekeyed accordingly with the JetSubstructurePacker. The slimming is done upstream on the pat::Jets made for the substructure algorithms, and so downstream this is no longer necessary to re-slim. \r\n\r\nThe jet content is (on 9000 events without pileup):\r\n\r\npatJets_slimmedJets__PAT\t1.00\t1.00\t1.00\t\t4.8%\r\npatJets_packedPatJetsAK8__PAT\t1.00\t0.58\t0.58\t\t2.8%\r\nrecoGenJets_slimmedGenJets__PAT\t1.00\t0.46\t0.46\t\t2.2%\r\nrecoGenJets_slimmedGenJetsAK8__PAT\t1.00\t0.45\t0.45\t\t2.2%\r\npatJets_slimmedJetsCMSTopTagCHSPacked_SubJets_PAT\t1.00\t0.16\t0.16\t\t0.8%\r\npatJets_slimmedJetsAK8PFCHSSoftDropPacked_SubJets_PAT\t1.00\t0.14\t0.14\t\t0.7%\r\n\r\nSo the additional collections (the subjets) are about 1.5% of the total miniAOD size. \r\n\r\n====== CPU usage =====\r\nThe additional CPU usage here is negligible because the workflow is not running any algorithms, but instead taking the values directly from the RECO collections upstream and rekeying the information : \r\n\r\nTimeReport   0.000074     0.000074     0.000074  caTopTagInfos\r\nTimeReport   0.000041     0.000041     0.000041  caTopTagInfosPAT\r\nTimeReport   0.000100     0.000100     0.000100  slimmedGenJets\r\nTimeReport   0.000128     0.000128     0.000128  slimmedGenJetsAK8\r\nTimeReport   0.000131     0.000131     0.000131  slimmedJets\r\nTimeReport   0.000000     0.000000     0.000000  slimmedJetsAK8\r\nTimeReport   0.000041     0.000041     0.000041  slimmedJetsAK8PFCHSSoftDropPacked\r\nTimeReport   0.000051     0.000051     0.000051  slimmedJetsAK8PFCHSSoftDropSubjets\r\nTimeReport   0.000048     0.000048     0.000048  slimmedJetsCMSTopTagCHSPacked\r\nTimeReport   0.000056     0.000056     0.000056  slimmedJetsCMSTopTagCHSSubjets\r\nTimeReport   0.000095     0.000095     0.000095  packedPatJetsAK8\r\n\r\n\r\n==== Testing ====\r\nTests are checking out (thanks to @ianna  for providing the correct recipe so quickly, I've merged in that PR, and apologies to @ferencek since I had to revert out the genJetsNoNu for the b-tagging sequence, to be discussed at an upcoming JMAR meeting). \r\n\r\n\r\n\r\n\r\n==== For posterity, here is the original PR message : ====\r\n\r\nWith @arizzi and @ferencek : we've added substructure into miniAOD in a slightly more natural way to allow for multiple interpretations of the jet. This will now be accessed through the \"subjet\" methods. They can take string or integer access similar to the JECs. I have also adjusted pt thresholds so that they are sensible given common user preferences. This makes the total of the AK8 jets (including two subjet collections) 3.8% of the total miniAOD size.\r\n\r\nThe nice additions are :\r\n\r\nSubjets from soft drop and the CMS top tagger are stored as lists of pat::Jets.\r\nJet corrections are applied to subjets.\r\nSubjet b-tagging is being run."},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/73895279","html_url":"https://github.com/cms-sw/cmssw/pull/7659#issuecomment-73895279","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7659","id":73895279,"user":{"login":"rappoccio","id":4267705,"avatar_url":"https://avatars.githubusercontent.com/u/4267705?v=3","gravatar_id":"","url":"https://api.github.com/users/rappoccio","html_url":"https://github.com/rappoccio","followers_url":"https://api.github.com/users/rappoccio/followers","following_url":"https://api.github.com/users/rappoccio/following{/other_user}","gists_url":"https://api.github.com/users/rappoccio/gists{/gist_id}","starred_url":"https://api.github.com/users/rappoccio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rappoccio/subscriptions","organizations_url":"https://api.github.com/users/rappoccio/orgs","repos_url":"https://api.github.com/users/rappoccio/repos","events_url":"https://api.github.com/users/rappoccio/events{/privacy}","received_events_url":"https://api.github.com/users/rappoccio/received_events","type":"User","site_admin":false},"created_at":"2015-02-11T15:04:05Z","updated_at":"2015-02-11T15:04:05Z","body":"@arizzi : I'm not sure I follow. The daughters of the AK8 jets are slimmed already (they're just pointers to the packed candidates). The subjets are accessible via the new \"subjets\" method in pat::Jet, and those in turn are pat::Jets that also have constituents pointing to the packed candidates. The subjets are slimmed (I implemented that yesterday at your suggestion) so that only the relevant ones are kept (i.e. pointed to by a jet that passes the pt cuts). "}}