cms-jet / PFNano

NanoAOD framework for testing jet algorithms.
7 stars 54 forks source link

Btag v2 #10

Closed andrzejnovak closed 4 years ago

andrzejnovak commented 4 years ago
alefisico commented 4 years ago

For reference:

andrzejnovak commented 4 years ago

@alefisico cool, it looks like adding SV info is fairly cheap storage-wise

alefisico commented 4 years ago

I dont have any major comment about this. If @rappoccio @laurenhay also dont have major comment, we can merge it to the master.

andrzejnovak commented 4 years ago

@rappoccio I've marked some of the SV variables if they are jet related or independent. I am not sure I understood exactly how you went about splitting the variables for PF cands, but if you have a moment later to do that splitting for the SVs as well that would be great. Even though this doesn't take up that much space.

rappoccio commented 4 years ago

@andrzejnovak The information for PFCandidates is all per-candidate here. There is additional information about track-->jet association here. Although, I'm not quite sure if that's what you mean.

andrzejnovak commented 4 years ago

@rappoccio Oh I think I finally understand. You create two separate tables for PF related info and add the cand index to link them, is that correct?

rappoccio commented 4 years ago

There's one table for PF candidates, and one table for variables that depend on both PF candidates AND the jet.