eic / epic-analysis

General (SI)DIS analysis framework for the EIC
GNU Lesser General Public License v3.0
3 stars 9 forks source link

ci: test `hepmc` files from Ralf through Delphes #194

Closed c-dilks closed 1 year ago

c-dilks commented 2 years ago

Briefly, what does this PR introduce?

Test new hepmc files from Ralf. Since these files are very large, we truncate them in the workflow config with mc head, so we don't overload runners.

The CI runs these files through Delphes, using the ATHENA card (we don't have an EPIC card yet). Thus in the _FULL_RESULTS artifact, the Delphes distributions (black, grey filled) should compare well with the ATHENA DeathValley data (red dashed).

What kind of change does this PR introduce?

Please check if this PR fulfills the following:

Does this PR introduce breaking changes? What changes might users need to make to their code?

No

Does this PR change default behavior?

It will change our Delphes HEPMC source under CI tests

c-dilks commented 2 years ago

18x275 noradcor looks okay

c-dilks commented 2 years ago

5x41 noradcor also looks OK; the pT distributions look closer to ECCE for some bins

c-dilks commented 2 years ago
c-dilks commented 2 years ago

CI checks on this branch will fail, since S3 files are now hepmc.gz format. See instead #197.