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

Add producers for tau ID and trigger weights. #4

Merged
merged 11 commits into from
Dec 23, 2022

Conversation

riga
Copy link
Member

@riga riga commented Dec 19, 2022

This PR adds two new producers that handle the creating of

  • Tau ID weights, and
  • trigger weights.

The trigger weights also cover the combination of single and di-tau triggers as measured by the TAU POG. Both implementations are based on the correctionlib. Uncertainties are propagated to weights according to https://twiki.cern.ch/twiki/bin/view/CMS/TauIDRecommendationForRun2?rev=113.

Note There are two commits that should have not gone into this feature branch. The file important for the review is production/tau.py.

@riga riga added the enhancement New feature or request label Dec 19, 2022
@riga riga requested a review from pkausw December 19, 2022 18:28
@riga riga self-assigned this Dec 19, 2022
hbt/config/configs_run2ul.py Show resolved Hide resolved
hbt/config/configs_run2ul.py Show resolved Hide resolved
hbt/production/tau.py Show resolved Hide resolved
hbt/production/tau.py Show resolved Hide resolved
hbt/production/tau.py Show resolved Hide resolved
@riga riga merged commit 91b2292 into master Dec 23, 2022
@riga riga deleted the feature/tau_and_trigger_weights branch December 23, 2022 09:05
aalvesan pushed a commit that referenced this pull request Jan 31, 2024
Add _nano_v9 postfix to current campaigns.
riga added a commit to riga/hh2bbtautau that referenced this pull request Jul 11, 2024
* Continue generalizing dy process ids.

* Simplification.

* Typo.

* Typo.
riga added a commit that referenced this pull request Jul 18, 2024
* added dy_process_ids producer und dy datasets

* forgot the linting

* fixed njet variable

* linting

* Continue generalizing dy process ids. (#4)

* Continue generalizing dy process ids.

* Simplification.

* Typo.

* Typo.

* define stiching ranges with aux dict

* typo

* Full draft for dy identifier.

* Simplify njets lookup, improve typing.

* Rename variables.

* update cf

* Update cf.

* Use stiched producer.

* Update cf.

* Update cf.

---------

Co-authored-by: haddadanas <hhhaaanas@gmail.com>
Co-authored-by: Anas <103462379+haddadanas@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants