-
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
Multiple RelVals: Unknown trigger name
exception
#44895
Comments
assign alca |
New categories assigned: alca @saumyaphor4252,@perrotta,@consuegs you have been requested to review this Pull request/Issue and eventually sign? Thanks |
cms-bot internal usage |
A new Issue was created by @iarspider. @smuzaffar, @rappoccio, @makortel, @antoniovilela, @Dr15Jones, @sextonkennedy can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
Also RelVal 1041 is failing in CMSSW_14_1_X_2024-05-02-1100 with the same exception. And RelVal 1040.1 is failing with the exception
I suppose the issue is surfaced by the merging in that IB of #44767 , which throws a clear exception if OutputModule SelectEvents refers to a non-existent Path or Process. The three failing workflows refer to 'RunExpressPhy2017F' (ExpressPhysics dataset for 2017). As far as I can read from https://twiki.cern.ch/twiki/bin/viewauth/CMS/AlCaRecoMatrix, in Run2 there was not such a wf 1040.1 was introduced in #22455: my impression is that a configuration for 2018 was (wrongly) used for a 2017 AlCaReco: unfortunately, the logs of the tests cannot be recovered to verify whether any message was issued in them for the missing path. The same for wf 1041.0 that was introduced (as previous 1040.2) with #33215, and wf 1042.0 that was introduced with #35631, but in both cases still with a configuration that does not seem to me to reflect the 2017 ALCARECO content. This is mostly to tell that I am investigating. Let me add here @francescobrivio (as author of some of the PRs that considered the Express content for 2017), my fellow @cms-sw/alca-l2 , and @cms-sw/ctpps-dpg-l2 for a clarification about the PPS content in Express for the 2017. And let me also add @makortel as author of #44767: did you notice Matti that after the CMSSW_14_1_X_2024-05-02-1100 the exception is not 'clear' any more (see the very same workflows failing in CMSSW_14_1 2024-05-02-2300)?. Do you understand why? |
We discovered several bugs in #44767 which are now fixed in #44891. These bugs resulted in the assert sometimes going off rather than the exception message being printed. We did extensive testing with the new fix and now see the exception message being issued consistently. |
Let's use this issue to report all the "unknown trigger names" from #44891 (comment) Workflow 1001.3
1040.1
1041.0, 1042.0
|
Unknown trigger paths
exception
Unknown trigger paths
exceptionUnknown trigger name
exception
My private test of 1001.3 on a single thread gives also
so all these workflows might have more issues than reported so far. |
#44914 should fix. |
RelVal 1042.0 is failing in multiple IBs with the following exception:
full log
The text was updated successfully, but these errors were encountered: