Skip to content
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

Non-reproducibility in TriggerResults in 24834.0, 24834.911, and 25034.999 #43790

Open
makortel opened this issue Jan 25, 2024 · 11 comments
Open

Comments

@makortel
Copy link
Contributor

It seems like there is a recent non-reproducibility in the TriggerResults comparisons of the step2 of the workflows 24834.0, 24834.911, and 25034.999 (all of them use D98). Here is an example from #43758 (comment)
image

Here is another occurrence #43738 (comment)

@makortel
Copy link
Contributor Author

assign upgrade, hlt

@cmsbuild
Copy link
Contributor

New categories assigned: upgrade,hlt

@Martin-Grunewald,@mmusich,@srimanob,@subirsarkar you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 25, 2024

cms-bot internal usage

@cmsbuild
Copy link
Contributor

A new Issue was created by @makortel Matti Kortelainen.

@antoniovilela, @smuzaffar, @rappoccio, @makortel, @sextonkennedy, @Dr15Jones can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@makortel
Copy link
Contributor Author

@mmusich
Copy link
Contributor

mmusich commented Jan 26, 2024

Given the pattern of merges between CMSSW_14_0_X_2024-01-22-100 (not visible) and CMSSW_14_0_X_2024-01-23-2300 (visible), see the gitHub diff of commits merged, it looks plausible that the non-reproducibility could come from #43738 (merged in CMSSW_14_0_X_2024-01-23-2300). Incidentally it was also noticed in the PR tests themselves: #43738 (comment).
@SohamBhattacharya @rovere FYI.

@makortel
Copy link
Contributor Author

makortel commented Feb 8, 2024

Is anyone looking into this non-reproducibility?

@dan131riley
Copy link

Possibly related to #43723? The fix for that issue in #43852 was just merged, so will be interesting to see if it improves the reproducibility.

@makortel
Copy link
Contributor Author

Possibly related to #43723? The fix for that issue in #43852 was just merged, so will be interesting to see if it improves the reproducibility.

The random differences are still there.

@makortel
Copy link
Contributor Author

makortel commented May 6, 2024

Is anyone looking into this non-reproducibility?

@mmusich
Copy link
Contributor

mmusich commented Oct 7, 2024

Is anyone looking into this non-reproducibility?

while I don't think anyone looked at this explicitly, I don't seem to be able to find recent instances (at least since ~ August 2024).
Perhaps it got fixed in one of the recent phase-2 HLT menus updates?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants