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
Update TriggerIdLoose definition to recover the efficiency in low-pT (backport to 10_1_X) #23125
Update TriggerIdLoose definition to recover the efficiency in low-pT (backport to 10_1_X) #23125
Conversation
A new Pull Request was created by @KyeongPil-Lee for CMSSW_10_1_X. It involves the following packages: DataFormats/MuonReco @perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
based on the tests already completed in the master branch, this PR changes the content of RECO and miniAOD objects. This is not allowed in the production release. An exception will need to be discussed. |
We are using offline selection online now. Are you suggesting that we have to move all that code under HLT to avoid this policy? We are changing meaning of one bit. |
On 5/4/18 12:30 AM, drkovalskyi wrote:
We are using offline selection online now. Are you suggesting that we
have to move all that code under HLT to avoid this policy? We are
changing meaning of one bit.
If this flag is only useful for online
an easy way to decouple is to add a config parameter and enable it in HLT.
If it's still needed in the offline files, then perhaps some versioning
is needed,
otherwise you are not going to be able to change the implementation of
the selector during the year.
we are still early enough in 10_1_X that an exception is possible both
for data and for MC.
|
It's expected that it will get stable as most muon selectors are and will be used for long time. At the moment we are still in integration phase because only now TSG is putting it all into the menu. A typical use case offline would be to check the id to get an estimated performance of the trigger id where it makes sense to do it. Getting it all right is too much work and we don't have manpower to support it or any need to do it. |
from the discussion in the RECO meeting, I expect that it should be possible to include this PR within the policy constraints due to other significant changes that have to go to the release. |
We will need to have more detailed instructions how to handle offline for online for future changes. |
the only clear solution is a configurable one; the selector for online may have to be versioned or reimplemented in the HLT producer if/when it's different. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1
This PR affects outputs both for the standard MC and data. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_1_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_10_2_X is complete. This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Backport of #23124