{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/10197","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/10197/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/10197/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/10197/events","html_url":"https://github.com/cms-sw/cmssw/pull/10197","id":94963683,"number":10197,"title":"Combined MVA computer based on TMVA (CombinedMVAV2)","user":{"login":"jpata","id":69717,"avatar_url":"https://avatars.githubusercontent.com/u/69717?v=3","gravatar_id":"","url":"https://api.github.com/users/jpata","html_url":"https://github.com/jpata","followers_url":"https://api.github.com/users/jpata/followers","following_url":"https://api.github.com/users/jpata/following{/other_user}","gists_url":"https://api.github.com/users/jpata/gists{/gist_id}","starred_url":"https://api.github.com/users/jpata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpata/subscriptions","organizations_url":"https://api.github.com/users/jpata/orgs","repos_url":"https://api.github.com/users/jpata/repos","events_url":"https://api.github.com/users/jpata/events{/privacy}","received_events_url":"https://api.github.com/users/jpata/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/analysis-pending","name":"analysis-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fef2c0"},{"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-pending","name":"tests-pending","color":"fbca04"}],"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":95,"closed_issues":384,"state":"open","created_at":"2015-06-23T07:27:04Z","updated_at":"2015-08-14T17:36:09Z","due_on":null,"closed_at":null},"comments":85,"created_at":"2015-07-14T14:52:16Z","updated_at":"2015-08-14T18:47:26Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/10197","html_url":"https://github.com/cms-sw/cmssw/pull/10197","diff_url":"https://github.com/cms-sw/cmssw/pull/10197.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/10197.patch"},"body":"This PR is to start the discussion on the inclusion of the combined MVA computer based on TMVA, as requested by the BTV conveners.\r\n\r\nI would appreciate comments on the implementation already now, however I still need to\r\n\r\n* update the weight files with another retraining\r\n* switch to GBRForest in TMVAEvaluator (probably done in common with BTV)\r\n* test on miniAOD."},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/131207281","html_url":"https://github.com/cms-sw/cmssw/pull/10197#issuecomment-131207281","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/10197","id":131207281,"user":{"login":"jpata","id":69717,"avatar_url":"https://avatars.githubusercontent.com/u/69717?v=3","gravatar_id":"","url":"https://api.github.com/users/jpata","html_url":"https://github.com/jpata","followers_url":"https://api.github.com/users/jpata/followers","following_url":"https://api.github.com/users/jpata/following{/other_user}","gists_url":"https://api.github.com/users/jpata/gists{/gist_id}","starred_url":"https://api.github.com/users/jpata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpata/subscriptions","organizations_url":"https://api.github.com/users/jpata/orgs","repos_url":"https://api.github.com/users/jpata/repos","events_url":"https://api.github.com/users/jpata/events{/privacy}","received_events_url":"https://api.github.com/users/jpata/received_events","type":"User","site_admin":false},"created_at":"2015-08-14T18:47:26Z","updated_at":"2015-08-14T18:47:26Z","body":"now I pushed the missing commits which remove the mutex lock and add the GBR flags. In the rebase, I didn't have unrelated modules checked out, previously this has worked fine.What is the recommended way to test with the HEAD of CMSSW_7_6_X branch?"}}