-
Notifications
You must be signed in to change notification settings - Fork 42
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
feat: add utility tool to validate histogram contents against a reference #149
Conversation
Example usage: python validate_histograms.py --reference reference/histos_1_file_per_process.json python validate_histograms.py --dump-json | jq
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.
Thanks! This is going to be very helpful. Given that we'll want to add more references for different settings and also will pick up some new systematics that will cause histograms to change, could you perhaps also add a script example for the JSON reference generation?
That would be just Is that what you mean? I'm not sure where to put that other than the |
Perfect, I hadn't looked closely enough at the code yet. No need to change anything then! |
Can we add a description of this tool to the README for cms-open-data-ttbar? |
@ekauffma sure, done! |
A reference file for the configuration with N_FILES_MAX_PER_SAMPLE=1 is provided as well.
The tool:
name;1
andname;2
)Fixes #136 .