Skip to content
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

implement PlotPostfitShapes task #69

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Conversation

mafrahm
Copy link
Collaborator

@mafrahm mafrahm commented Jan 31, 2024

There still are some TODOs for a future PR:
- include data
- include correct uncertainty bands
- pass correct binning information

How to create postfit shapes:

  1. Create your rebinned datacards (can be skipped since this is required from step 2)
law run hbw.ModifyDatacardsFlatRebin --version prod1 --inference-model rates_only
  1. Get the command on how to run FitDiagnostics (this ensures that we keep the category name information when merging datacards)
law run hbw.PrepareInferenceTaskCalls --version prod1 --inference-model rates_only
  1. Step 2 prints a task call. Copy-paste this and run this with Inference (needs a fresh shell). Run it again with --print-output 0 to get the output location of our FitDiagnostics file.
cd path/to/inference
source setup.sh
<insert_copied_task_call>
<insert_copied_task_call> --print-output 0
  1. Run the PlotPostfitShapes task using the FitDiagnostics file from before. This task also knows some plotting parameters like yscale and shape-norm
law run hbw.PlotPostfitShapes --version prod1 --inference-model rates_only --fit-diagnostics-file <file_name>

@mafrahm mafrahm self-assigned this Jan 31, 2024
@mafrahm mafrahm marked this pull request as ready for review February 5, 2024 12:09
@mafrahm mafrahm requested a review from Lara813 February 5, 2024 12:09
@mafrahm mafrahm merged commit 5f42e44 into master Feb 5, 2024
4 checks passed
@mafrahm mafrahm deleted the feature/postfit_plots branch February 5, 2024 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants