richstu / nano2pico

Code to produce analysis ntuples (pico) from the CMS NanoAOD.
2 stars 4 forks source link

nano2pico minor bug fixes #50

Closed jdgrassi closed 1 month ago

jdgrassi commented 2 months ago

y pT, y jet overlap, wgt sums float bug, trigger pT branches. Ready for code review

mcoshiro commented 2 months ago

Checked code over and things look good to me. Also checked that it runs after a clean checkout and the output looks correct, so from my side, it looks good to merge

andersbarzdukas commented 2 months ago

Checked out code and ran it. Output looks good to me. Code changes look good as well. I think it also looks good to merge

andersbarzdukas commented 1 month ago

If there are not more comments today I will merge.

I will then quickly add Run 3 MC datasets and a minor change to IsoTrack array length that causes a crash in Run 3 MC.