-
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
Exception in ExternalLHEProducer after merging #40939 #41230
Comments
assign generators |
A new Issue was created by @perrotta Andrea Perrotta. @Dr15Jones, @perrotta, @dpiparo, @rappoccio, @makortel, @smuzaffar can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
New categories assigned: generators @mkirsano,@menglu21,@alberto-sanchez,@SiewYan,@GurpreetSinghChahal,@Saptaparna you have been requested to review this Pull request/Issue and eventually sign? Thanks |
Hi, sorry, this issue appears to be because this line of regex: https://github.com/cms-sw/cmssw/blob/master/GeneratorInterface/LHEInterface/scripts/mergeLHE.py#L259 catches the new LHE |
Ah, it seems there is a further issue only for the Herwig workflows that the order in which the lhe numbers are read is different than I would expect. I'll need to explore it some more, but I think this could take a little while to resolve |
Thank you Dominic. |
Yes, since I don't have an estimate for how long this will take to fix, it's probably best to revert the PR, if that's relatively easy to do from your side, then I'll make a new PR once I have a fix |
I have prepared two PRs to revert #40939 (reverted by #41237) and cms-sw/cmsdist#8349 (reverted by cms-sw/cmsdist#8417). They can be merged once tested succesfully in the IB. |
Thank you, and sorry for not catching these earlier |
PRs reverted for next CMSSW_13_1_X_2023-03-31-1100 |
Hi @perrotta A similar issue is also observed for Summer22 production (with CMSSW_12_4_11_patch3) for the MCFM samples i.e. https://cms-unified.web.cern.ch/cms-unified/showlog/?search=task_HIG-Run3Summer22EEwmLHEGS-00173 |
@sunilUIET Please open a new issue as the cause is very likely different ( |
Since the merging of #40939 several workflows are crashing in the IBs with the following error message (e.g. from wf 512.0):
Please notice that the very same error appears independently on the merge of cms-sw/cmsdist#8409, which I forgot for CMSSW_13_1_X_2023-03-29-1100 and merged only later on for CMSSW_13_1_X_2023-03-29-2300, but as you can see both IBs are crashing with the same error message.
Author @Dominic-Stafford has been informed.
A fix is needed, otherwise we will be eventually forced to revert PR #40939 and the accompanying cmsdist one.
The text was updated successfully, but these errors were encountered: