{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/7584","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7584/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7584/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7584/events","html_url":"https://github.com/cms-sw/cmssw/pull/7584","id":56726577,"number":7584,"title":"Use rapidity instead of pseudo-rapidity in the packing of PackedGenParticles","user":{"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},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/analysis-approved","name":"analysis-approved","color":"009800"},{"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/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","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":49,"closed_issues":659,"state":"open","created_at":"2014-11-21T15:23:58Z","updated_at":"2015-02-13T09:51:38Z","due_on":"2015-01-26T08:00:00Z","closed_at":null},"comments":10,"created_at":"2015-02-05T20:43:05Z","updated_at":"2015-02-13T09:57:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/7584","html_url":"https://github.com/cms-sw/cmssw/pull/7584","diff_url":"https://github.com/cms-sw/cmssw/pull/7584.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/7584.patch"},"body":"It was noted that in existing miniAOD some GenJets have invalid refs to PackedGenParticles that extend in eta beyond 6. These particles cannot be saved because the range for eta in the packing is restricted to [-6,6]. \r\nOn the other hand while eta of a GenJet constituent can extend beyond 6, the \"rapidity\" (used in jet clustering) cannot extend beyond GenJetEta + JetDRparameter. Considering relevant jets up to eta 5, the constituents should never exceed 6 (even for AK8 GenJets we should get up to 5.8).\r\n\r\nThe PackedGenCandidates are then fixed by storing persistently the \"rapidity\" in range [-6,6] rather than the pseudorapidity.\r\nAn ioread function is provided to keep compatibility (i.e. be able to read 72X MINIAOD)\r\n\r\nSpeed at read-time should be checked to see if this introduces any regression.\r\n\r\n@gpetruc \r\n"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/74230114","html_url":"https://github.com/cms-sw/cmssw/pull/7584#issuecomment-74230114","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7584","id":74230114,"user":{"login":"cmsbuild","id":4170616,"avatar_url":"https://avatars.githubusercontent.com/u/4170616?v=3","gravatar_id":"","url":"https://api.github.com/users/cmsbuild","html_url":"https://github.com/cmsbuild","followers_url":"https://api.github.com/users/cmsbuild/followers","following_url":"https://api.github.com/users/cmsbuild/following{/other_user}","gists_url":"https://api.github.com/users/cmsbuild/gists{/gist_id}","starred_url":"https://api.github.com/users/cmsbuild/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmsbuild/subscriptions","organizations_url":"https://api.github.com/users/cmsbuild/orgs","repos_url":"https://api.github.com/users/cmsbuild/repos","events_url":"https://api.github.com/users/cmsbuild/events{/privacy}","received_events_url":"https://api.github.com/users/cmsbuild/received_events","type":"User","site_admin":false},"created_at":"2015-02-13T09:57:41Z","updated_at":"2015-02-13T09:57:41Z","body":"This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @degano, @ktf, @smuzaffar"}}