-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Implementation of TauWPThreshold without need to catch exceptions #43890
Implementation of TauWPThreshold without need to catch exceptions #43890
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-43890/38748
|
A new Pull Request was created by @mbluj for master. It involves the following packages:
@mandrenguyen, @jfernan2, @cmsbuild can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
FYI, @kandrosov, @fwyzard, @mmusich |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d49615/37279/summary.html Comparison SummarySummary:
|
@mbluj @kandrosov thanks for the quick fix 👍🏻 |
Shoud it be backported? Which release series are going to be used to test and then run HLT in 2024? |
@mbluj, minimally, please backport it to CMSSW_14_0_X (which is the current operations release this year). CMSSW_13_3_X would be good for internal consistency, but not strictly necessary. |
+reconstruction |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @rappoccio, @sextonkennedy, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
type tau |
+1 |
PR description:
As title says, this PR implements parsing of threshold values in the
TauWPThreshold
class using function which does not require catching exceptions. Namely it replaces usage ofstd::stod
bystd::strtod
.It addresses #39627 independently raised also here #43100 (comment).
Q: I guess it should be backported - to which release series?
PR validation:
Checked with a custom workflow that new implementation of
TauWPThreshold
behaves as expected for simple numbers, expressions and incorrect expressions.