Read edm4hep

[andresailer]

BEGINRELEASENOTES

• DDSim: add a reader for EDM4hep files, fixes Create EDM4hep reader for DDG4 #1369
  • DDSim can now read EDM4hep files in root format.

ENDRELEASENOTES

• Actually ingest parameters
  • [x] Also ingest GeneratorInfo (to be done later Add GeneratorInfo to EDM4hepReader #1373)
• Test with complicated events

[github-actions]

Test Results

   14 files     14 suites   5h 26m 27s ⏱️
  370 tests   370 ✅ 0 💤 0 ❌
2 539 runs  2 539 ✅ 0 💤 0 ❌

Results for commit 5906930.

♻️ This comment has been updated with latest results.

[andresailer]

That GeneratorInfo is not yet written to EDM4hep files?

[tmadlener]

That GeneratorInfo is not yet written to EDM4hep files?

Not consistently, i.e. if at all currently only with files that are produced via k4GeneratorsConfig. So I think we have to consider that effectively optional at this point, until that has become more prevalent. Additionally, there are still some changes pending, so maybe we should consider postponing that here in any case?

[tmadlener]

This will inadvertently mix "file level" metadata and run parameters for the output, right?

[andresailer]

Indeed it would do that.

[tmadlener]

Is there a way to keep them separated? E.g. by attaching the metadata frame to some entity as an extension and then writing it out again as a whole in the output if it is present?