Skip to content

Remove old repeated variables, update configurations#89

Open
aebid wants to merge 16 commits into
cms-flaf:mainfrom
aebid:Remove_Dead_Vars
Open

Remove old repeated variables, update configurations#89
aebid wants to merge 16 commits into
cms-flaf:mainfrom
aebid:Remove_Dead_Vars

Conversation

@aebid

@aebid aebid commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Some cleaning of repeated variables. Includes reorganizing histogram configs and removing old vars, updating DNN configs to use unified var names, updating global to plot unified var names.

@aebid

aebid commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

@cms-flaf-bot please test

@cms-flaf-bot

Copy link
Copy Markdown

pipeline#15036711 started

@cms-flaf-bot

Copy link
Copy Markdown

pipeline#15036711 passed

@aebid

aebid commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

@kandrosov this PR is ready for review.

It changes:
Analysis/hh_bbww.py variables to be more uniform in new naming scheme obj_obj_function, as well as cleaning up variable definition locations and removing old unused variables.
Then adds many new histogram configurations for new variables, and adds these variables in the normal variable list in global.yaml.
Comments out the bbWW DY correction as it is seen to be bad, and leaves the bbtautau DY correction as default.
Last it changes the DNN configurations to use the new variable names, and updates the era configurations for new plotting colors, signal scaling, and SR_mbb naming.

@aebid aebid changed the title Remove old repeated variables Remove old repeated variables, update configurations Jun 15, 2026
@aebid

aebid commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

@cms-flaf-bot please test

  • StatInference_version=PR_18

@cms-flaf-bot

Copy link
Copy Markdown

pipeline#15057240 started

@cms-flaf-bot

Copy link
Copy Markdown

pipeline#15057240 failed

@aebid

aebid commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Actually the CI cannot test meta eras as each node only has one era available. So this statInference PR cannot be tested with the CI.

@aebid

aebid commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

@cms-flaf-bot please test

@cms-flaf-bot

Copy link
Copy Markdown

pipeline#15057452 started

@cms-flaf-bot

Copy link
Copy Markdown

pipeline#15057452 passed

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