Closed iarspider closed 5 days ago
assign RecoHGCal/TICL
New categories assigned: reconstruction,upgrade
@jfernan2,@mandrenguyen,@Moanwar,@srimanob,@subirsarkar 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.
@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.
cms-bot commands are listed here
@felicepantaleo , could this be due to https://github.com/cms-sw/cmssw/pull/46651 change?
I'll produce a larger statistics for these HFNose workflows and get back to you.
type HFNose
(not sure this type exists)
I suspect this is due to this mask being filled only inside the if statement:
https://github.com/cms-sw/cmssw/blob/master/RecoHGCal/TICL/plugins/TrackstersProducer.cc#L205-L210
To make sure I'll reproduce locally and apply the fix and make a PR.
I was able to reproduce and test the fix in #46709
@smuzaffar can we add 32034.0 to the wf tested in all PRs? It's a good wf to check whether we are breaking something inside the TICL core framework.
can we add 32034.0 to the wf tested in all PRs?
if it's useful in general (even outside of PR tests, but also something that developers are recommended to check before submitting PRs),
is a good place to start.
@felicepantaleo , sure if this is useful to catch errors earlier then as @mmusich mentioned we can either add it in to short/limited matrix set or add it to https://github.com/cms-sw/cms-bot/blob/master/cmssw-pr-test-config#L11C23-L11C29 for bot PR tests only. Feel to open PR for any one of these
Relval 32034.0 fails with SIGSEGV on step 3 in multiple IBs, e.g. link