EDM4hep Discussion

Tuesday 23 Jun 2020, 09:00 → 10:00 Europe/Zurich

Vidyo

Live Notes

EDM4hep Live Notes
==================

Date: June 23, 2020
Indico: https://indico.cern.ch/event/931971/

This is a document for taking notes during EDM4hep meetings.

Connected: Thomas, Andre, Benedikt, Placido, Xingtao, Clement, Valentin, Weidong, Frank, Jiaheng, Gerri

Apologies: Tao

## Introduction and General Points

 

## Progress and Discussion

### Delphes plugin

* Update June 23:
    * PR Ready for review
        * https://github.com/key4hep/EDM4hep/pull/56
        * Depends on some other PRs
            * (Optional) Non-const getter: https://github.com/AIDASoft/podio/pull/95
            * See notes below
            * (Optional) Make mass member: https://github.com/key4hep/EDM4hep/pull/69
    * Important information converted
    * Jet Variables not yet
    * Converter class available
    * Muons/Electrons stored as "RecoParticleRef"
        * Like the Subset collections in LCIO
        * Issue/WIP: Can only read one muon back
    * WIP: Documentation for Configuration options

* https://github.com/key4hep/EDM4hep/pull/56
    * https://github.com/AIDASoft/podio/pull/95
* Some remaining technical questions:
    * Is there always one collection with MCParticles?
        - Delphes can have stable, all
    * Convenient accessor for Metadata information would be nice
        * E.g.: particle ID
        * -> LCIO::ParticleIDNavigator convenience class
            * http://lcio.desy.de/v02-09/doc/doxygen_api/html/classUTIL_1_1PIDHandler.html (?)
    * List of all Reconstructed Particles from Delphes?
    * How to find track associated to Reconstructed Particle?
        * Only via related MCParticle
    

* What should the Delphes outputfile look like?
    1) Standard Delphes output
    2) Output looking like full reconstruction output

*  Links to existing plugins etc:
    * https://github.com/HEP-FCC/FCCSW/blob/master/Sim/SimDelphesInterface/src/
    * Overwrite EventStore functions from podio
        * Storing different Jet variables for example
        * Feed them back to podio later
    * Finalize conversion of reconstructed particles
    * started delphes2lcio:
        * https://github.com/iLCSoft/LCIO/pull/80

### Spack installation

* Nightly build should be working again after PR
    * Link:
* Depending on spack-based LCG releases
    * Need discuss with Ivan how to proceed

### Linking issues with K4FWcore/EDM4hep
* https://github.com/key4hep/K4FWCore/issues/9
    * Should be fixed by correct name in rootmap file
        * https://github.com/AIDASoft/podio/pull/99
        * https://github.com/key4hep/EDM4hep/pull/68
    * Root is looking for rootmaps in LD_LIBRARY_PATH
        * Document requirements for them to be there

After using the latest PODIO and EDM4hep, the I/O problem solved.

### Nightly Builds and CI

## Podio

### Non-const getter

* https://github.com/AIDASoft/podio/pull/95
* Need to understand use cases and desired behaviour from podio
* Alternative is using a `const_cast` as in dd4hep output

### Meta Data

#### Usage of "metadata" for user defined data

### Issues following MetaData Developments

* Some issues with podio, Frank will open issues, create (failing) tests
* Issue: https://github.com/AIDASoft/podio/issues/103
    * Test: https://github.com/AIDASoft/podio/pull/102

#### Writing second file with another tree
* Related to https://github.com/AIDASoft/podio/issues/103 ?
* Cannot write second file?
* Have to do: ``` TFile::Open("tmp.root", "RECREATE")``` ?
    * Doesn't help

### EventStore

- Needs some reworking and definition of interfaces

### Schema Evolution

- Version for object descriptions, etc.
- Open issue: https://github.com/AIDASoft/podio/issues/86

### Features

* Subset collections?

## LCIOConverters

* Used to reconstruct clusters with Pandora
    * Publish Pandora Interface on github

 

## EDM4hep

### Release 1.0

* Need:
    * ~~Plugin~~
    * ~~Eventheader~~
    * ~~Meta Data (Event / Run Parameters)~~

### Units convention?
* What's the default units?
    * Comments in the data model, same as LCIO
        * mm and GeV and (angles)
* Should we have several utilities to convert the units from Geant4, DD4hep, ROOT to EDM4hep?
    * Geant4, Root, DD4hep can (should) use the same units from newest(?) versions (MeV and mm)
        * Check edm4hep output from dd4hep
        * https://github.com/key4hep/EDM4hep/blob/8078d059b2a14da571ddc29fd6baf52de045420e/plugins/DDG4EDM4hep/edm4hep/Geant4Output2EDM4hep.cpp#L445
    * Probably don't need additional conversion?

### Reconstructed Particle: Mass dependent on energy?
* https://github.com/key4hep/EDM4hep/pull/69
* See explanations in PR
* Keep separate members, but also allow consistent setting?
* Add utility to return 4-vector from Reconstructed Particle
    * Ignore the mass?
* Clarify consistency responsibility to user?
* Need more time to think about this

## AOB

### Dual Readout Calorimeter Plans
* First implement DD4hep description
* Look at use of EDM4hep for simulation, different data model in use at the moment

### Next meeting:

* July 7, 2020

 

 

09:00 → 09:05 Introduction

Speakers: Andre Sailer (CERN), Frank-Dieter Gaede (Deutsches Elektronen-Synchrotron (DE))

EDM4hep Github

Podio Github

09:55 → 10:00 Discussion

Speaker: Dr All