search

cmstas / NanoTools

tools, usage examples for CMS nanoAOD
1 stars 15 forks source link

move abs() #2

Closed aminnj closed 4 years ago

aminnj commented 4 years ago

Spotted this just now. Since there's only electrons and muons, this isn't a bug here, but should be fixed for consistency (also see if there are any similar places that could actually be bugged.

https://github.com/cmstas/NanoTools/blob/b1a90dc21617ad01c7d629a2439a58a2d4c21472/NanoCORE/IsolationTools.cc#L9-L13

(abs should only wrap id)

jkguiang commented 4 years ago

Fixed here: https://github.com/cmstas/NanoTools/commit/b2470c04764d6ce90051a9b2da58e5bed0de6fb7