{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/23738","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/23738/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/23738/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/23738/events","html_url":"https://github.com/cms-sw/cmssw/pull/23738","id":337916541,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk4OTc2NTg4","number":23738,"title":"Improvements and fixes for the BPH-trigger offline DQM code","user":{"login":"aboletti","id":12882645,"node_id":"MDQ6VXNlcjEyODgyNjQ1","avatar_url":"https://avatars3.githubusercontent.com/u/12882645?v=4","gravatar_id":"","url":"https://api.github.com/users/aboletti","html_url":"https://github.com/aboletti","followers_url":"https://api.github.com/users/aboletti/followers","following_url":"https://api.github.com/users/aboletti/following{/other_user}","gists_url":"https://api.github.com/users/aboletti/gists{/gist_id}","starred_url":"https://api.github.com/users/aboletti/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aboletti/subscriptions","organizations_url":"https://api.github.com/users/aboletti/orgs","repos_url":"https://api.github.com/users/aboletti/repos","events_url":"https://api.github.com/users/aboletti/events{/privacy}","received_events_url":"https://api.github.com/users/aboletti/received_events","type":"User","site_admin":false},"labels":[{"id":673935047,"node_id":"MDU6TGFiZWw2NzM5MzUwNDc=","url":"https://api.github.com/repos/cms-sw/cmssw/labels/code-checks-approved","name":"code-checks-approved","color":"2cbe4e","default":false},{"id":88653795,"node_id":"MDU6TGFiZWw4ODY1Mzc5NQ==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fbca04","default":false},{"id":52820549,"node_id":"MDU6TGFiZWw1MjgyMDU0OQ==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/dqm-pending","name":"dqm-pending","color":"fbca04","default":false},{"id":56953091,"node_id":"MDU6TGFiZWw1Njk1MzA5MQ==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04","default":false},{"id":53167770,"node_id":"MDU6TGFiZWw1MzE2Nzc3MA==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/pending-signatures","name":"pending-signatures","color":"ff8000","default":false},{"id":56131195,"node_id":"MDU6TGFiZWw1NjEzMTE5NQ==","url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-approved","name":"tests-approved","color":"2cbe4e","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/76","html_url":"https://github.com/cms-sw/cmssw/milestone/76","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/76/labels","id":3227631,"node_id":"MDk6TWlsZXN0b25lMzIyNzYzMQ==","number":76,"title":"CMSSW_10_2_X","description":null,"creator":{"login":"smuzaffar","id":4115138,"node_id":"MDQ6VXNlcjQxMTUxMzg=","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":48,"closed_issues":692,"state":"open","created_at":"2018-03-29T07:57:10Z","updated_at":"2018-07-04T14:36:29Z","due_on":null,"closed_at":null},"comments":15,"created_at":"2018-07-03T14:09:47Z","updated_at":"2018-07-04T14:48:26Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/23738","html_url":"https://github.com/cms-sw/cmssw/pull/23738","diff_url":"https://github.com/cms-sw/cmssw/pull/23738.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/23738.patch"},"body":"In this pull-request many features of the offline DQM code used for BPH triggers have been fixed and optimised, and few more functionalities have been added.\r\nHere a list of the major changes:\r\n\r\n- **New general structure** of the code: for each efficiency type (defined as the \"cases\" in the big \"switch\") the numerator histos are filled in a nested environment just after the denominator filling, instead of defining two separate environments with specific selection criteria for each of them.\r\n- **Fixed trigger matching** feature: the `bool BPHMonitor::matchToTrigger` function is now using the HLTConfigProvider method to access the trigger-object collections.\r\n- **New correction for prescaled triggers**: while using the reference method to produce efficiencies, if the numerator path is prescaled it would flatly decrease the resulting efficiency by that prescale value. By filling the numerator histograms with weighted events, according to that prescale, the reduction effect is corrected.\r\n- **Better tuning of the offline selections** (especially the dimuon invariant masses), to select the correct regions where to compute the efficiency. This helps in some modules to select a pure region for the signal, in other modules (based on background events) to select the appropriate kinematic region.\r\n- **Customised and optimised binning**: to reduce the global number of bins, especially where they are not needed, the binning of the histograms has been refined\r\n- **Removal of few meaningless efficiencies**: some modules defined in the list of the cff file (devoted to the L1 efficiency of dR(MuMu) cut in 2mu+trk triggers) have been removed, since their efficiency definition is not correct (using at denominator a 2mu trigger, they will always be biased by the inefficiency of the track reconstruction at HLT)"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/402500260","html_url":"https://github.com/cms-sw/cmssw/pull/23738#issuecomment-402500260","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/23738","id":402500260,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMjUwMDI2MA==","user":{"login":"cmsbuild","id":4170616,"node_id":"MDQ6VXNlcjQxNzA2MTY=","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-07-04T14:48:26Z","updated_at":"2018-07-04T14:48:26Z","author_association":"CONTRIBUTOR","body":"Comparison is ready\nhttps://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23738/28998/summary.html\n\nComparison Summary:\n - No significant changes to the logs found\n - Reco comparison results: 0 differences found in the comparisons\n - DQMHistoTests: Total files compared: 31\n - DQMHistoTests: Total histograms compared: 2893480\n - DQMHistoTests: Total failures: 13\n - DQMHistoTests: Total nulls: 0\n - DQMHistoTests: Total successes: 2893277\n - DQMHistoTests: Total skipped: 190\n - DQMHistoTests: Total Missing objects: 0\n - DQMHistoSizes: Histogram memory added: -1451.104 KiB( 30 files compared)\n - DQMHistoSizes: changed ( 10824.0,... ): -90.694 KiB      HLT/BPH\n - Checked 128 log files, 14 edm output root files, 31 DQM output files\n"}}
{"actor":{"display_login":"cmsbuild"}}