cmsb2g / B2GAnaFW

Analysis framework for Beyond Two Generations (B2G) Physics Analysis Group (PAG) of the Compact Muon Solenoid (CMS) Experiment
8 stars 33 forks source link

New tag for 2017 data #80

Open dmajumder opened 7 years ago

dmajumder commented 7 years ago

Starting a new issue to track the developments for a tag for 2017 data using 92X release. Can we have the experts update the latest recipes: @alefisico @skyriacoCMS @jkarancs @lbenato May be we start with the JetToolBox recipe in CMSSW_9_2_X?

alefisico commented 7 years ago

do we have any timescale for this? just to know since I am very busy currently on other stuff

dmajumder commented 7 years ago

@alefisico the timescale is as usual "ASAP" because we are planning to adapt this package to monitor triggers. See my reply to Lisa's email on the B2G HN: https://hypernews.cern.ch/HyperNews/CMS/get/b2g/1055/1/1.html

If the JetToolbox in CMSSW_9_1_X works out of the box in CMSSW_9_2_X then it is very straightforward. I ran with the CMSSW_9_1_X JetToolBox in CMSSW_9_2_6 and at least it did not crash, but we have to update a lot of stuff I think. Is there anybody else who can help with the development?

alefisico commented 7 years ago

that is main problem, it does not. There were some changes in the unscheduled mode that changes everything in the jetToolbox. I'll try to work on them asap, but I have some other deadlines now.

dmajumder commented 7 years ago

indeed, I think there is a Task module than now needs to be used. Can you specify a time line (no pressure)? In the meanwhile, I can at least try to make the other objects work in the workflow.

skyriacoCMS commented 7 years ago

Hello all,

I have a random question. Why do the met variables get stored as vectors and not as simple floats in our ntuple? Shouldn't it be a single number per event?

Cheers,

Savvas Kyriacou Department of Physics and Astronomy Rutgers, The State University of New Jersey

CERN B8 R-020 ,__, (o.o) /),,,,) " "

On Thu, Jul 13, 2017 at 3:51 PM, dmajumder notifications@github.com wrote:

indeed, I think there is a Task module than now needs to be used. Can you specify a time line (no pressure)? In the meanwhile, I can at least try to make the other objects work in the workflow.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cmsb2g/B2GAnaFW/issues/80#issuecomment-315083831, or mute the thread https://github.com/notifications/unsubscribe-auth/AHCcD7opo0P622jZ3pF97r_-2GCK-NGEks5sNiDlgaJpZM4OVabZ .

dmajumder commented 7 years ago

@skyriacoCMS probably it is historical. I agree we can change the VPSet to PSet.

alefisico commented 7 years ago

@dmajumder Is there any target release? any CMSSW_9_2_X? About the jetToolbox, there is an easy (and temporary?) fix that should work now. I can check it with the B2GAnaFwk now, although it should be transparent to the framework. also, can you create a 92X working branch?

dmajumder commented 7 years ago

@alefisico the target cmssw release should be the latest CMSSW_9_2_X. I have started working in CMSSW_9_2_6 which I think is the most recent. New development branch created https://github.com/cmsb2g/B2GAnaFW/tree/CMSSW_9_2_X_V0

alefisico commented 7 years ago

ok, the code runs and including the latest version of the toolbox: https://github.com/cms-jet/JetToolbox/tree/jetToolbox_91X_v1 BUT, the output file is empty.. @dmajumder do you have the same?