{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/7548","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7548/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7548/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7548/events","html_url":"https://github.com/cms-sw/cmssw/pull/7548","id":56435439,"number":7548,"title":"Ecalmultifit use DB inputs and retuning of spike flagging","user":{"login":"emanueledimarco","id":4517974,"avatar_url":"https://avatars.githubusercontent.com/u/4517974?v=3","gravatar_id":"","url":"https://api.github.com/users/emanueledimarco","html_url":"https://github.com/emanueledimarco","followers_url":"https://api.github.com/users/emanueledimarco/followers","following_url":"https://api.github.com/users/emanueledimarco/following{/other_user}","gists_url":"https://api.github.com/users/emanueledimarco/gists{/gist_id}","starred_url":"https://api.github.com/users/emanueledimarco/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/emanueledimarco/subscriptions","organizations_url":"https://api.github.com/users/emanueledimarco/orgs","repos_url":"https://api.github.com/users/emanueledimarco/repos","events_url":"https://api.github.com/users/emanueledimarco/events{/privacy}","received_events_url":"https://api.github.com/users/emanueledimarco/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/alca-approved","name":"alca-approved","color":"009800"},{"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/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","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":37,"closed_issues":556,"state":"open","created_at":"2014-11-21T15:23:58Z","updated_at":"2015-02-04T18:07:38Z","due_on":"2015-01-26T08:00:00Z","closed_at":null},"comments":11,"created_at":"2015-02-03T20:53:24Z","updated_at":"2015-02-04T18:20:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/7548","html_url":"https://github.com/cms-sw/cmssw/pull/7548","diff_url":"https://github.com/cms-sw/cmssw/pull/7548.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/7548.patch"},"body":"This pull request contains:\r\n\r\n1) use of the DB inputs in the multifit local reconstruction. For the time being, the GT uses the MC conditions for both data and MC, so no change is expected wrt the previous one (python configuration with MC conditions)\r\n\r\n2) re-introduction of the spike killing flags, kOutOfTime and kWeird. The cuts for the topological variables have been reoptimised. If those flags are used in the PF rechits, there could be some minimal change wrt previous version.\r\n\r\n3) autocond files contain the GTs with the payloads (there should be the same commit by Salvatore @diguida on the AlCa DB side).\r\n\r\nAdding @bendavid @argiro @cerminar @diguida to check this.\r\n\r\nIMPORTANT: for @bachtis : this version has the kOutOfTime flag re-enabled (for spikes flagging). Please, if you want, disable it in the PF clustering filtering"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/72908483","html_url":"https://github.com/cms-sw/cmssw/pull/7548#issuecomment-72908483","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/7548","id":72908483,"user":{"login":"emanueledimarco","id":4517974,"avatar_url":"https://avatars.githubusercontent.com/u/4517974?v=3","gravatar_id":"","url":"https://api.github.com/users/emanueledimarco","html_url":"https://github.com/emanueledimarco","followers_url":"https://api.github.com/users/emanueledimarco/followers","following_url":"https://api.github.com/users/emanueledimarco/following{/other_user}","gists_url":"https://api.github.com/users/emanueledimarco/gists{/gist_id}","starred_url":"https://api.github.com/users/emanueledimarco/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/emanueledimarco/subscriptions","organizations_url":"https://api.github.com/users/emanueledimarco/orgs","repos_url":"https://api.github.com/users/emanueledimarco/repos","events_url":"https://api.github.com/users/emanueledimarco/events{/privacy}","received_events_url":"https://api.github.com/users/emanueledimarco/received_events","type":"User","site_admin":false},"created_at":"2015-02-04T18:20:30Z","updated_at":"2015-02-04T18:20:30Z","body":"if @bachtis confirms that this is the only place it occurs, I will do the change and update the pull-request. In this way we have the flagging ready to be used in case we need it, but it is turned off by default in PF."}}