Skip to content

Branch 14 10 2014#2

Merged
xshi merged 5 commits intocms-bph:masterfrom
gechen:branch-14-10-2014
Oct 14, 2014
Merged

Branch 14 10 2014#2
xshi merged 5 commits intocms-bph:masterfrom
gechen:branch-14-10-2014

Conversation

@gechen
Copy link
Copy Markdown
Contributor

@gechen gechen commented Oct 14, 2014

trkpt & trkdcabs; KeepGENOnly; Jpsi & Psi(2S)
@xshi @nsahoo

@nsahoo
Copy link
Copy Markdown
Contributor

nsahoo commented Oct 14, 2014

Hi @gechen,
Thanks for the update, here are my comments.

please change the preselection cut values of trkpt and trkdcasigbs, from your slides last week, it seems ok if you do the following changes
trackpt 0.1 ----> 0.2 GeV
trackdcasigbs 0.0 ----> 0.1
as a result, the ntuple size and CPU time will be less.

There is "NO" need of defining two scripts btokmumu_2012_cfi.py and btokmumu_2012_cfi_GENOnly.py...The only difference I can see is
KeepGENOnly = cms.untracked.bool(False),
You can add it to https://github.com/gechen/BToKMuMu/blob/branch-14-10-2014/python/btokmumu_2012_cfi.py#L175

@nsahoo
Copy link
Copy Markdown
Contributor

nsahoo commented Oct 14, 2014

👍

@gechen
Copy link
Copy Markdown
Contributor Author

gechen commented Oct 14, 2014

OK!

@gechen gechen closed this Oct 14, 2014
@gechen gechen reopened this Oct 14, 2014
@gechen
Copy link
Copy Markdown
Contributor Author

gechen commented Oct 14, 2014

This branch cann't merge to cms-bph/BToKMuMu????but gechen/BToKMuMu!!!!

@nsahoo
Copy link
Copy Markdown
Contributor

nsahoo commented Oct 14, 2014

Yes, it can be merged...Xin can only do that..I don't have the access to merge pull requests.

xshi added a commit that referenced this pull request Oct 14, 2014
@xshi xshi merged commit 3fef448 into cms-bph:master Oct 14, 2014
xshi pushed a commit that referenced this pull request Oct 22, 2014
@gechen gechen deleted the branch-14-10-2014 branch October 27, 2014 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants