{"action":"created","issue":{"url":"https://api.github.com/repos/alexpearce/CharmProduction/issues/53","labels_url":"https://api.github.com/repos/alexpearce/CharmProduction/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/alexpearce/CharmProduction/issues/53/comments","events_url":"https://api.github.com/repos/alexpearce/CharmProduction/issues/53/events","html_url":"https://github.com/alexpearce/CharmProduction/pull/53","id":56121379,"number":53,"title":"Add stripping lines","user":{"login":"alexpearce","id":540853,"avatar_url":"https://avatars.githubusercontent.com/u/540853?v=3","gravatar_id":"","url":"https://api.github.com/users/alexpearce","html_url":"https://github.com/alexpearce","followers_url":"https://api.github.com/users/alexpearce/followers","following_url":"https://api.github.com/users/alexpearce/following{/other_user}","gists_url":"https://api.github.com/users/alexpearce/gists{/gist_id}","starred_url":"https://api.github.com/users/alexpearce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexpearce/subscriptions","organizations_url":"https://api.github.com/users/alexpearce/orgs","repos_url":"https://api.github.com/users/alexpearce/repos","events_url":"https://api.github.com/users/alexpearce/events{/privacy}","received_events_url":"https://api.github.com/users/alexpearce/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/alexpearce/CharmProduction/labels/selection","name":"selection","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-01-31T10:59:22Z","updated_at":"2015-03-04T17:34:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/alexpearce/CharmProduction/pulls/53","html_url":"https://github.com/alexpearce/CharmProduction/pull/53","diff_url":"https://github.com/alexpearce/CharmProduction/pull/53.diff","patch_url":"https://github.com/alexpearce/CharmProduction/pull/53.patch"},"body":"I'm including these in this repo so we can open issues on them and stuff. The changes in this branch haven't been included in [StrippingSelections](https://svnweb.cern.ch/trac/lhcb/browser/Stripping/trunk/Phys/StrippingSelections/python/StrippingSelections) yet, I'll wait until we have proto-lines for all modes. Which brings me too…\r\n\r\n### Stripping lines for all modes\r\n\r\nThis is the list we need to complete to have stripping lines for everything we want.\r\n\r\n- [x] [D0Tohh](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD02HHForXSec.py)\r\n- [x] [D0ToKpipipi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD02K3PiForXSec.py)\r\n- [x] [DpToKpipi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD2HHHForXSec.py)\r\n- [x] [DpToKKpi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD2HHHForXSec.py) (or [DpTophipi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD2PhiPiForXSec.py))\r\n- [x] [DsTophipi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD2PhiPiForXSec.py)\r\n- [x] [DsTopipipi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD2HHHForXSec.py)\r\n- [x] [Dst0ToD0pi_D0Tohh](https://github.com/alexpearce/CharmProduction/blob/f10b648945d95c3bd02ebd480b37f0994c3c7fde/stripping/StrippingD02HHForXSec.py)\r\n- [x] [Dst0ToD0pi_D0ToKpipipi](https://github.com/alexpearce/CharmProduction/blob/f10b648945d95c3bd02ebd480b37f0994c3c7fde/stripping/StrippingD02K3PiForXSec.py)\r\n- [x] [DstToD0pi_D0Tohh](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD02K3PiForXSec.py)\r\n- [x] [DstToD0pi_D0ToKpipipi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingD02K3PiForXSec.py)\r\n- [x] [LcTopKpi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingLambdac2PKPiForXSec.py)\r\n- [x] [LcTopKK](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingLambdac2PKPiForXSec.py) (or [LcToppipi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingLambdac2PKPiForXSec.py))\r\n- [x] [ScToLcpi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingLambdac2PKPiForXSec.py)\r\n- [x] [Sc0ToLcpi](https://github.com/alexpearce/CharmProduction/blob/1da82bbfbe4c436899272931a660688b33ca796c/stripping/StrippingLambdac2PKPiForXSec.py)\r\n- [x] [XicTopKpi](https://github.com/alexpearce/CharmProduction/blob/66608f50a6a5a1e9d96e709ac97e37565b4d4e7f/stripping/StrippingXic2PKPiForXSec.py)\r\n- [x] [XicTopKKpi](https://github.com/alexpearce/CharmProduction/blob/66608f50a6a5a1e9d96e709ac97e37565b4d4e7f/stripping/StrippingXic2PKKPiForXSec.py)\r\n\r\n(Better suggestions for shorthand names?)\r\n\r\nThe D\\*0 modes can go in the respective D0 lines, should be very similar selections to the D\\*+ modes already in them. The Xi_c lines should have a new module. (We could capture the pKpi mode in the Lambda_c module by extending the mass window, but the mass difference is [quite](http://pdg8.lbl.gov/rpp2014v1/pdgLive/Particle.action?node=S033) [large](http://pdg8.lbl.gov/rpp2014v1/pdgLive/Particle.action?node=S045) so we'd probably get an unacceptable rate. Makes most sense to me to just duplicate the Lambda_c module and edit it.)\r\n\r\n### Selections\r\n\r\nI haven't altered the selections at all, other than changing [`ADOCAMAX`](https://twiki.cern.ch/twiki/bin/view/LHCb/LoKiArrayFunctions#LoKiAMAXDOCA) to [`ACUTDOCA`](https://twiki.cern.ch/twiki/bin/view/LHCb/LoKiHybridFilters#AMAXDOCA_AMINDOCA_Distance_of_cl) as apparently it's better for CPU usage. I've checked this by running each stripping line other 10k events before and after the alterations, then `diff`'ing the resulting two log files. I didn't see any changes in the number of selected candidates, only differences in timing (where fluctuations are expected), so concluded the modifications haven't changed the physics.\r\n\r\nBut still, we need to look at the current selections and update them. They are very consistent across the lines: the same sets of cuts, often with the same values. I think we should strive for something similar as it will make our lives easier."},"comment":{"url":"https://api.github.com/repos/alexpearce/CharmProduction/issues/comments/77204999","html_url":"https://github.com/alexpearce/CharmProduction/pull/53#issuecomment-77204999","issue_url":"https://api.github.com/repos/alexpearce/CharmProduction/issues/53","id":77204999,"user":{"login":"alexpearce","id":540853,"avatar_url":"https://avatars.githubusercontent.com/u/540853?v=3","gravatar_id":"","url":"https://api.github.com/users/alexpearce","html_url":"https://github.com/alexpearce","followers_url":"https://api.github.com/users/alexpearce/followers","following_url":"https://api.github.com/users/alexpearce/following{/other_user}","gists_url":"https://api.github.com/users/alexpearce/gists{/gist_id}","starred_url":"https://api.github.com/users/alexpearce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexpearce/subscriptions","organizations_url":"https://api.github.com/users/alexpearce/orgs","repos_url":"https://api.github.com/users/alexpearce/repos","events_url":"https://api.github.com/users/alexpearce/events{/privacy}","received_events_url":"https://api.github.com/users/alexpearce/received_events","type":"User","site_admin":false},"created_at":"2015-03-04T17:34:00Z","updated_at":"2015-03-04T17:34:00Z","body":"Hmm, how about checking the key defining the weights file then?\r\n\r\n```python\r\ndef makeMVASelection(self, name, inputSel):\r\n    if not self.config['D0_MVA_Weights']:\r\n        return inputSel\r\n    # ...\r\n```"}}