-
Notifications
You must be signed in to change notification settings - Fork 120
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
Setup speed and more deterministic behavior for jetmet tools #484
Conversation
lgray
commented
Apr 3, 2021
•
edited
edited
- make instantiating corrected jets faster at the cost of a longer evaluation time
- make instantiating corrected met faster at the cost of a longer evaluation time
- use per-batch deterministic random seed for jet smearing ( fix Deterministic jet smearing #454 )
- investigate and fix reported memory leak in met corrector ( fixed in Actually fix MET cycles #487 )
- investigate and fix Output of FactorizedJetCorrector.getCorrection has too many dimensions #461
@JayjeetAtGithub can you see what's going on with this failure of the rados tests? It looks like there's something screwey on the docker side of things. It's not failing in anything related to coffea. |
@kondratyevd if you have time - give this a quick look. The instantiation speed is an order of magnitude faster. |
@lgray Yeah, I am trying to find the problem |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My only real worry is possible cycles, some extra tests or avoiding using the user-supplied cache for the more macroscopic virtual arrays (e.g. the delayed unc. builder) would be good
2e2599d
to
7311d3e
Compare