{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/9130","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/9130/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/9130/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/9130/events","html_url":"https://github.com/cms-sw/cmssw/pull/9130","id":77620060,"number":9130,"title":"Adding a DQM module for dEdxHit info (74X)","user":{"login":"quertenmont","id":4639944,"avatar_url":"https://avatars.githubusercontent.com/u/4639944?v=3","gravatar_id":"","url":"https://api.github.com/users/quertenmont","html_url":"https://github.com/quertenmont","followers_url":"https://api.github.com/users/quertenmont/followers","following_url":"https://api.github.com/users/quertenmont/following{/other_user}","gists_url":"https://api.github.com/users/quertenmont/gists{/gist_id}","starred_url":"https://api.github.com/users/quertenmont/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quertenmont/subscriptions","organizations_url":"https://api.github.com/users/quertenmont/orgs","repos_url":"https://api.github.com/users/quertenmont/repos","events_url":"https://api.github.com/users/quertenmont/events{/privacy}","received_events_url":"https://api.github.com/users/quertenmont/received_events","type":"User","site_admin":false},"labels":[{"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/dqm-pending","name":"dqm-pending","color":"fbca04"},{"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/reconstruction-pending","name":"reconstruction-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","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_7_4_X","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":52,"closed_issues":1078,"state":"open","created_at":"2014-11-21T15:23:58Z","updated_at":"2015-05-19T21:28:20Z","due_on":"2015-01-26T08:00:00Z","closed_at":null},"comments":16,"created_at":"2015-05-18T11:32:18Z","updated_at":"2015-05-19T23:57:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/9130","html_url":"https://github.com/cms-sw/cmssw/pull/9130","diff_url":"https://github.com/cms-sw/cmssw/pull/9130.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/9130.patch"},"body":"This PR add a DQM module to monitor dEdxHit info.\r\nAnd also fix a small bug in the dEdxHitInfo producer in standard RECO.\r\nHope this can be included in the release that will be used for both MC and Data reconstruction\r\nBackport of this changes for 75X are also there"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/103700266","html_url":"https://github.com/cms-sw/cmssw/pull/9130#issuecomment-103700266","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/9130","id":103700266,"user":{"login":"slava77","id":4676718,"avatar_url":"https://avatars.githubusercontent.com/u/4676718?v=3","gravatar_id":"","url":"https://api.github.com/users/slava77","html_url":"https://github.com/slava77","followers_url":"https://api.github.com/users/slava77/followers","following_url":"https://api.github.com/users/slava77/following{/other_user}","gists_url":"https://api.github.com/users/slava77/gists{/gist_id}","starred_url":"https://api.github.com/users/slava77/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/slava77/subscriptions","organizations_url":"https://api.github.com/users/slava77/orgs","repos_url":"https://api.github.com/users/slava77/repos","events_url":"https://api.github.com/users/slava77/events{/privacy}","received_events_url":"https://api.github.com/users/slava77/received_events","type":"User","site_admin":false},"created_at":"2015-05-19T23:57:43Z","updated_at":"2015-05-19T23:57:43Z","body":"\r\nThe nominal policy is NOT to fix issues that change physics output in a major way in a production release where a major campaign has started.\r\nIn the case of 74X, the major MC campaign has started.  \r\nSo, if the object is buggy, it will remain buggy consistently between data and MC.\r\n\r\nRealistically, \r\n- for 50 ns data, the  RECO data tier will be available and the newly introduced object is not really necessary\r\n- MC will not be redone, except for private processing\r\n\r\nIf this product had sense only for data (and not MC), then we could apply the fix.\r\nThe story appears to be the opposite: there should be a meaningful solution for 50 ns data (RECO),\r\nwhile the fix is really needed in MC (made only in AODSIM)."}}