{"action":"created","issue":{"url":"https://api.github.com/repos/cp3-llbb/Framework/issues/211","repository_url":"https://api.github.com/repos/cp3-llbb/Framework","labels_url":"https://api.github.com/repos/cp3-llbb/Framework/issues/211/labels{/name}","comments_url":"https://api.github.com/repos/cp3-llbb/Framework/issues/211/comments","events_url":"https://api.github.com/repos/cp3-llbb/Framework/issues/211/events","html_url":"https://github.com/cp3-llbb/Framework/pull/211","id":182055965,"number":211,"title":"Split b-tagging SFs for light and heavy flavors","user":{"login":"blinkseb","id":386274,"avatar_url":"https://avatars.githubusercontent.com/u/386274?v=3","gravatar_id":"","url":"https://api.github.com/users/blinkseb","html_url":"https://github.com/blinkseb","followers_url":"https://api.github.com/users/blinkseb/followers","following_url":"https://api.github.com/users/blinkseb/following{/other_user}","gists_url":"https://api.github.com/users/blinkseb/gists{/gist_id}","starred_url":"https://api.github.com/users/blinkseb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blinkseb/subscriptions","organizations_url":"https://api.github.com/users/blinkseb/orgs","repos_url":"https://api.github.com/users/blinkseb/repos","events_url":"https://api.github.com/users/blinkseb/events{/privacy}","received_events_url":"https://api.github.com/users/blinkseb/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cp3-llbb/Framework/labels/CMSSW%208.0.X","name":"CMSSW 8.0.X","color":"006b75"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-10-10T16:17:41Z","updated_at":"2016-10-11T08:31:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cp3-llbb/Framework/pulls/211","html_url":"https://github.com/cp3-llbb/Framework/pull/211","diff_url":"https://github.com/cp3-llbb/Framework/pull/211.diff","patch_url":"https://github.com/cp3-llbb/Framework/pull/211.patch"},"body":"We should treat light and heavy flavors b-tagging SFs as uncorrelated. To achieve this, this PR split the SFs branch for b-tagging into two, one for light flavors and one for heavy flavors. If a jet is light, the heavy branch is filled with a 1 +- 0 SF, and vice versa. This allow one to use the heavy SFs for every jet without knowing if it's actually a light or a heavy jet. In case this information is needed, a new branch is added, `jet_systFlavor`, which can be either 1 for heavy or 2 for light jets.\r\n\r\nI run it and it works as expected. All SFs from the heavy branch are 1 for light jets (and the opposite), and sane values are present for light & heavy jets.\r\n\r\n~~There's also an interface change in the `get_scale_factor` method, which now expects a `Flavor` argument, needed to decide between the light & heavy SFs.~~ No longer relevant\r\n\r\nTODO:\r\n - [x] Update reference trees"},"comment":{"url":"https://api.github.com/repos/cp3-llbb/Framework/issues/comments/252846658","html_url":"https://github.com/cp3-llbb/Framework/pull/211#issuecomment-252846658","issue_url":"https://api.github.com/repos/cp3-llbb/Framework/issues/211","id":252846658,"user":{"login":"blinkseb","id":386274,"avatar_url":"https://avatars.githubusercontent.com/u/386274?v=3","gravatar_id":"","url":"https://api.github.com/users/blinkseb","html_url":"https://github.com/blinkseb","followers_url":"https://api.github.com/users/blinkseb/followers","following_url":"https://api.github.com/users/blinkseb/following{/other_user}","gists_url":"https://api.github.com/users/blinkseb/gists{/gist_id}","starred_url":"https://api.github.com/users/blinkseb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blinkseb/subscriptions","organizations_url":"https://api.github.com/users/blinkseb/orgs","repos_url":"https://api.github.com/users/blinkseb/repos","events_url":"https://api.github.com/users/blinkseb/events{/privacy}","received_events_url":"https://api.github.com/users/blinkseb/received_events","type":"User","site_admin":false},"created_at":"2016-10-11T08:31:50Z","updated_at":"2016-10-11T08:31:50Z","body":"Updated dropping the `s` from strings :wink: I also added a new method in `JetProducer` to avoid changing the interface for analyses. It's now transparent."}}
{"actor":{"display_login":"blinkseb"}}