{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/11093","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11093/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11093/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11093/events","html_url":"https://github.com/cms-sw/cmssw/pull/11093","id":104594419,"number":11093,"title":"make EcalRingCalibrationTools thread safe (76x)","user":{"login":"fwyzard","id":4069793,"avatar_url":"https://avatars.githubusercontent.com/u/4069793?v=3","gravatar_id":"","url":"https://api.github.com/users/fwyzard","html_url":"https://github.com/fwyzard","followers_url":"https://api.github.com/users/fwyzard/followers","following_url":"https://api.github.com/users/fwyzard/following{/other_user}","gists_url":"https://api.github.com/users/fwyzard/gists{/gist_id}","starred_url":"https://api.github.com/users/fwyzard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fwyzard/subscriptions","organizations_url":"https://api.github.com/users/fwyzard/orgs","repos_url":"https://api.github.com/users/fwyzard/repos","events_url":"https://api.github.com/users/fwyzard/events{/privacy}","received_events_url":"https://api.github.com/users/fwyzard/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/alca-pending","name":"alca-pending","color":"fbca04"},{"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/hlt-approved","name":"hlt-approved","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/pending-signatures","name":"pending-signatures","color":"eb6420"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-rejected","name":"tests-rejected","color":"e11d21"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/55","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_7_6_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/55/labels","id":1176788,"number":55,"title":"Next CMSSW_7_6_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":85,"closed_issues":557,"state":"open","created_at":"2015-06-23T07:27:04Z","updated_at":"2015-09-03T11:26:08Z","due_on":null,"closed_at":null},"comments":13,"created_at":"2015-09-02T23:57:33Z","updated_at":"2015-09-03T11:55:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/11093","html_url":"https://github.com/cms-sw/cmssw/pull/11093","diff_url":"https://github.com/cms-sw/cmssw/pull/11093.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/11093.patch"},"body":"Implement a minimal set of changes to EcalRingCalibrationTools to make its initialisation and usage thead-safe.\r\nIn particular, this should make HLTEcalPhiSymFilter thread-safe again.\r\n\r\nNote that EcalRingCalibrationTools is still non-compliant with the CMS coding rules, and should be rewritten either as an ESProducer, or as a non-static data member of an EDProducer."},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/137416464","html_url":"https://github.com/cms-sw/cmssw/pull/11093#issuecomment-137416464","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/11093","id":137416464,"user":{"login":"davidlange6","id":5042883,"avatar_url":"https://avatars.githubusercontent.com/u/5042883?v=3","gravatar_id":"","url":"https://api.github.com/users/davidlange6","html_url":"https://github.com/davidlange6","followers_url":"https://api.github.com/users/davidlange6/followers","following_url":"https://api.github.com/users/davidlange6/following{/other_user}","gists_url":"https://api.github.com/users/davidlange6/gists{/gist_id}","starred_url":"https://api.github.com/users/davidlange6/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davidlange6/subscriptions","organizations_url":"https://api.github.com/users/davidlange6/orgs","repos_url":"https://api.github.com/users/davidlange6/repos","events_url":"https://api.github.com/users/davidlange6/events{/privacy}","received_events_url":"https://api.github.com/users/davidlange6/received_events","type":"User","site_admin":false},"created_at":"2015-09-03T11:55:54Z","updated_at":"2015-09-03T11:55:54Z","body":"Am I mistaken that computing the ringID from a DetId is more than a few lines of code? \r\n\r\n> On Sep 3, 2015, at 1:38 PM, Badder Marzocchi <notifications@github.com> wrote:\r\n> \r\n> @fwyzard Is an alternative still needed? An alternative tool in CMSSW for extracting the ringID is this one:\r\n> \r\n> https://github.com/cms-sw/cmssw/blob/CMSSW_7_5_X/Calibration/EcalCalibAlgos/src/EcalGeomPhiSymHelper.cc\r\n> \r\n> however it is not used and maintained anymore, so don't know if it works correctly with the latest CMSSW version.\r\n> \r\n> —\r\n> Reply to this email directly or view it on GitHub.\r\n> \r\n\r\n"}}