{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/16435","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16435/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16435/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16435/events","html_url":"https://github.com/cms-sw/cmssw/pull/16435","id":186851246,"number":16435,"title":"JER MET uncertainties and METSignifiance re-estimation","user":{"login":"mmarionncern","id":5114248,"avatar_url":"https://avatars.githubusercontent.com/u/5114248?v=3","gravatar_id":"","url":"https://api.github.com/users/mmarionncern","html_url":"https://github.com/mmarionncern","followers_url":"https://api.github.com/users/mmarionncern/followers","following_url":"https://api.github.com/users/mmarionncern/following{/other_user}","gists_url":"https://api.github.com/users/mmarionncern/gists{/gist_id}","starred_url":"https://api.github.com/users/mmarionncern/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmarionncern/subscriptions","organizations_url":"https://api.github.com/users/mmarionncern/orgs","repos_url":"https://api.github.com/users/mmarionncern/repos","events_url":"https://api.github.com/users/mmarionncern/events{/privacy}","received_events_url":"https://api.github.com/users/mmarionncern/received_events","type":"User","site_admin":false},"labels":[{"id":52881179,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/analysis-pending","name":"analysis-pending","color":"fbca04","default":false},{"id":88645479,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-available","name":"comparison-available","color":"009800","default":false},{"id":56953091,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04","default":false},{"id":53167770,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/pending-signatures","name":"pending-signatures","color":"eb6420","default":false},{"id":52820010,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/reconstruction-pending","name":"reconstruction-pending","color":"fbca04","default":false},{"id":56131195,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-approved","name":"tests-approved","color":"009800","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/59","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_8_1_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/59/labels","id":1590252,"number":59,"title":"Next CMSSW_8_1_X","description":"","creator":{"login":"smuzaffar","id":4115138,"avatar_url":"https://avatars.githubusercontent.com/u/4115138?v=3","gravatar_id":"","url":"https://api.github.com/users/smuzaffar","html_url":"https://github.com/smuzaffar","followers_url":"https://api.github.com/users/smuzaffar/followers","following_url":"https://api.github.com/users/smuzaffar/following{/other_user}","gists_url":"https://api.github.com/users/smuzaffar/gists{/gist_id}","starred_url":"https://api.github.com/users/smuzaffar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smuzaffar/subscriptions","organizations_url":"https://api.github.com/users/smuzaffar/orgs","repos_url":"https://api.github.com/users/smuzaffar/repos","events_url":"https://api.github.com/users/smuzaffar/events{/privacy}","received_events_url":"https://api.github.com/users/smuzaffar/received_events","type":"User","site_admin":false},"open_issues":69,"closed_issues":1896,"state":"open","created_at":"2016-02-18T09:12:12Z","updated_at":"2016-11-14T09:01:11Z","due_on":"2016-11-01T00:00:00Z","closed_at":null},"comments":27,"created_at":"2016-11-02T16:28:52Z","updated_at":"2016-11-14T10:04:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/16435","html_url":"https://github.com/cms-sw/cmssw/pull/16435","diff_url":"https://github.com/cms-sw/cmssw/pull/16435.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/16435.patch"},"body":"This PR enables the re-computation of the MET significance for the corrected METs on PAT, and enable the JER uncertainty propagation to the MET when the jets are not smeared.\r\n\r\nPackages touched:\r\n- PhysicsTools/PatAlgos\r\n- RecoMET/METAlgorithms\r\n\r\nFormat and content changes :\r\n- JER uncertainties for non-smeared not set to the central value anymore\r\n- central MET significance is not supposed to change when reprocessing AODs (i.e. in slimmedMET)\r\n\r\nNo real changes of performances/miniAOD size is expected. the MET significance changes only has an effect when the MET is recomputed on top of miniAOD, but the covariance matrix is not recomputed, which means that CPU ressources stays basically the same. The jet smearing used to compute the JER variation is small and does not reduce significantly the processing rate"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/260295114","html_url":"https://github.com/cms-sw/cmssw/pull/16435#issuecomment-260295114","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16435","id":260295114,"user":{"login":"mmarionncern","id":5114248,"avatar_url":"https://avatars.githubusercontent.com/u/5114248?v=3","gravatar_id":"","url":"https://api.github.com/users/mmarionncern","html_url":"https://github.com/mmarionncern","followers_url":"https://api.github.com/users/mmarionncern/followers","following_url":"https://api.github.com/users/mmarionncern/following{/other_user}","gists_url":"https://api.github.com/users/mmarionncern/gists{/gist_id}","starred_url":"https://api.github.com/users/mmarionncern/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmarionncern/subscriptions","organizations_url":"https://api.github.com/users/mmarionncern/orgs","repos_url":"https://api.github.com/users/mmarionncern/repos","events_url":"https://api.github.com/users/mmarionncern/events{/privacy}","received_events_url":"https://api.github.com/users/mmarionncern/received_events","type":"User","site_admin":false},"created_at":"2016-11-14T10:04:56Z","updated_at":"2016-11-14T10:04:56Z","body":"Hi Slava, sorry for the late answer : \r\n\r\n>>>> isn't JER scaling the standard central value for the jets? \r\n\r\nI am not sure I understand the comment, but if I take the statement literally, yes, it is basically doing that when we have access ot the genJet.\r\n\r\n>>>> Do we need to add uncertainties to the unified framework for the cases with \r\n>>>> - jets not corrected by residual corrections \r\n>>>> - jets not corrected by L123 (e.g. provide raw jets scaled by 1/JEC) ?\r\n\r\nI don't think so, people are not supposed to use raw jets. They can rederive such uncertainties by modifying the level of JEC correction in the MET tool, but I would let it as it is for experts only rather than making them accessible to everybody with an extra probability to use the wrong uncertainties."}}
{"actor":{"display_login":"mmarionncern"}}