{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/22696","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/22696/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/22696/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/22696/events","html_url":"https://github.com/cms-sw/cmssw/pull/22696","id":307320068,"number":22696,"title":"Switch to pfCandidates for b tagging for Puppi jets","user":{"login":"imarches","id":4941296,"avatar_url":"https://avatars0.githubusercontent.com/u/4941296?v=4","gravatar_id":"","url":"https://api.github.com/users/imarches","html_url":"https://github.com/imarches","followers_url":"https://api.github.com/users/imarches/followers","following_url":"https://api.github.com/users/imarches/following{/other_user}","gists_url":"https://api.github.com/users/imarches/gists{/gist_id}","starred_url":"https://api.github.com/users/imarches/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/imarches/subscriptions","organizations_url":"https://api.github.com/users/imarches/orgs","repos_url":"https://api.github.com/users/imarches/repos","events_url":"https://api.github.com/users/imarches/events{/privacy}","received_events_url":"https://api.github.com/users/imarches/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":673935047,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/code-checks-approved","name":"code-checks-approved","color":"2cbe4e","default":false},{"id":88653795,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fbca04","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":"ff8000","default":false},{"id":52820010,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/reconstruction-pending","name":"reconstruction-pending","color":"fbca04","default":false},{"id":56131152,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-pending","name":"tests-pending","color":"fbca04","default":false},{"id":530655448,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/upgrade-pending","name":"upgrade-pending","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/73","html_url":"https://github.com/cms-sw/cmssw/milestone/73","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/73/labels","id":3060828,"number":73,"title":"CMSSW_10_1_X","description":null,"creator":{"login":"smuzaffar","id":4115138,"avatar_url":"https://avatars3.githubusercontent.com/u/4115138?v=4","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":95,"closed_issues":509,"state":"open","created_at":"2018-01-24T11:24:59Z","updated_at":"2018-03-21T17:54:17Z","due_on":null,"closed_at":null},"comments":10,"created_at":"2018-03-21T16:25:30Z","updated_at":"2018-03-21T18:09:11Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/22696","html_url":"https://github.com/cms-sw/cmssw/pull/22696","diff_url":"https://github.com/cms-sw/cmssw/pull/22696.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/22696.patch"},"body":"Solution to the issue discussed here: https://github.com/cms-sw/cmssw/pull/22608\r\n\r\nSwitch back to standard pfCandidates for b tagging also for Puppi jets, reverting the change introduced in: https://github.com/cms-sw/cmssw/pull/18666\r\n\r\nThis change was introduced as it was giving some b tagging performance improvement for Phase2, which is not there anymore with the new Puppi cuts for Phase 2.\r\n\r\nGoing back to pfCandidates for b tagging recovers in a satisfactory way the performance as shown in the red curves below. It's also a safer choice as it does not expose b tagging to changes in the Puppi algorithm and its parameters.\r\n\r\n![image](https://user-images.githubusercontent.com/4941296/37722408-3e69aae2-2d2c-11e8-9541-3443b6fb906f.png)\r\n\r\n![image](https://user-images.githubusercontent.com/4941296/37722434-4d406db2-2d2c-11e8-8a02-e23b98041e37.png)\r\n\r\nThe performance of the red curves (standard b tagging with pfCandidates and Phase 2 Puppi cuts) is close now to the black curves, using special custom Phase 2 cuts, which was essentially the attempted solution in https://github.com/cms-sw/cmssw/pull/22608.\r\n\r\nThe PR can be backported to 93X and included in: https://github.com/cms-sw/cmssw/issues/22554\r\n"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/375042929","html_url":"https://github.com/cms-sw/cmssw/pull/22696#issuecomment-375042929","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/22696","id":375042929,"user":{"login":"cmsbuild","id":4170616,"avatar_url":"https://avatars1.githubusercontent.com/u/4170616?v=4","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":"2018-03-21T18:09:11Z","updated_at":"2018-03-21T18:09:11Z","author_association":"CONTRIBUTOR","body":"The tests are being triggered in jenkins."}}
{"actor":{"display_login":"cmsbuild"}}