Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'Soft' codes for lepton trigger & ID SF #121

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

quark2
Copy link
Contributor

@quark2 quark2 commented Feb 11, 2019

I made a loader of lepton trigger & ID SF root file. Using this, we can read the SF root files directly. Also, mergeSFroots.py merges SF root files if there are several SF root files for each periods (see [1]).

Currently, only single lepton channel part in Run2016 is contained. Data for Run2017 will be added someday. Btw, the root files (see d4531d3) are added for test, but if a pull request on nanoData (see [2]) is merged, then they will not be needed.

[1] https://github.com/CPLUOS/nanoData/blob/78b824f90bd9c259da71f3e9ac6cdeb3fb277ca6/scaleFactor/2016/README.md
[2] CPLUOS/nanoData#1

@tt8888tt tt8888tt added the Analyser-common Analyser related but change on common code (AFFECT TO OTHERS) label Mar 1, 2019
@watson-ij
Copy link
Contributor

From the nano meeting: Jason would prefer csv files to hold the scale-factors, rather than root files. The data should live in nanoData, and get pulled in at the same time as this PR. CPLUOS/nanoData#1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Analyser-common Analyser related but change on common code (AFFECT TO OTHERS)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants