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

Fix jet matching with Pythia Versions later than 8.3 #144

Merged
merged 5 commits into from
Dec 20, 2024

Conversation

bistapf
Copy link
Contributor

@bistapf bistapf commented Dec 19, 2024

BEGINRELEASENOTES

  • Add the missing else block to the DelphesPythia8Reader to make jet matching also work with Pythia Versions later than 8.3. Use same code as in the equivalent Delphes reader.

ENDRELEASENOTES

@tmadlener
Copy link
Contributor

Is there an easy way to check whether matching works as expected that could be put into a test case?

@bistapf
Copy link
Contributor Author

bistapf commented Dec 19, 2024

Do you mean to check if it's actually running/properly set-up, or also that it gives correct physics?

For the first, can just check if: number reco events < number input LHE events in a test case with matching turned on. Should always be the case if jet matching works correctly I think(note that only for some FCC-hh sample we need to use the matching, so need a dedicated tester job with one of these). And/or could check for this print-out at the end of the run:

 *-------  JetMatching Error and Warning Messages Statistics  -----------------------------------------------------* 
 |                                                                                                                 | 
 |  times   message                                                                                                | 
 |                                                                                                                 | 
 |      0   no errors or warnings to report                                                                        | 
 |                                                                                                                 | 
 *-------  End JetMatching Error and Warning Messages Statistics  -------------------------------------------------* 

Don't know if Pythia has some internal variable it uses for this that we could access/check.

To check also if physics makes sense, could compare the resulting matching efficiency of a test job to whatever this process had in a previous production (this is stored in the FCC database somewhere), but they are derived from very large scale productions so would be a rough comparison since it might vary a little depending which LHE file/seed you're running on.

@tmadlener
Copy link
Contributor

No, really just on the technical level to make sure if a pythia card with matching turned on, we actually get matching running. So checking the output is a solution, but we should make sure that the output was indeed different before this fix (and that the test actually fails before).

@tmadlener tmadlener enabled auto-merge (squash) December 20, 2024 10:38
@tmadlener tmadlener merged commit c8ad257 into key4hep:main Dec 20, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants