{"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/new-package-pending","name":"new-package-pending","color":"e11d21"},{"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-started","name":"tests-started","color":"eb6420"}],"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":78,"closed_issues":301,"state":"open","created_at":"2015-06-23T07:27:04Z","updated_at":"2015-08-04T13:51:09Z","due_on":null,"closed_at":null},"comments":28,"created_at":"2015-07-14T14:52:16Z","updated_at":"2015-08-04T16:47:52Z","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/127671661","html_url":"https://github.com/cms-sw/cmssw/pull/10197#issuecomment-127671661","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/10197","id":127671661,"user":{"login":"ferencek","id":4885289,"avatar_url":"https://avatars.githubusercontent.com/u/4885289?v=3","gravatar_id":"","url":"https://api.github.com/users/ferencek","html_url":"https://github.com/ferencek","followers_url":"https://api.github.com/users/ferencek/followers","following_url":"https://api.github.com/users/ferencek/following{/other_user}","gists_url":"https://api.github.com/users/ferencek/gists{/gist_id}","starred_url":"https://api.github.com/users/ferencek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ferencek/subscriptions","organizations_url":"https://api.github.com/users/ferencek/orgs","repos_url":"https://api.github.com/users/ferencek/repos","events_url":"https://api.github.com/users/ferencek/events{/privacy}","received_events_url":"https://api.github.com/users/ferencek/received_events","type":"User","site_admin":false},"created_at":"2015-08-04T16:47:52Z","updated_at":"2015-08-04T16:47:52Z","body":"I think it's fine to add xml files just for testing. The branch can be reset later or history rewritten to get rid of the xml files before merging. Merging, of course, should not happen before either the external files are added or we add GBRForest payloads to the global tag."}}