EDM4hep Discussion

Europe/Zurich
Vidyo

Vidyo

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

Date: May 26, 2020
Indico: https://indico.cern.ch/event/922741/

This is a document for taking notes during EDM4hep meetings.

Connected: Erica, Gerardo, Tao, Thomas, Andre, Jiaheng, Valentin, Clement, Placido, Frank, Graeme,

Apologies: Weidong

## Introduction and General Points

 

## Progress and Discussion


### Delphes plugin

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

### Spack installation

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

* gcc 8.3.1 (from devtoolset) does not have spack recipe
* Relocation issue, spack error, but works manually
* NB (added after the meeting): Compatibility with devtoolset and ROOT: https://root-forum.cern.ch/t/devtoolset-gcc-toolset-compatibility/38286
    * Just for devtoolset 3?


### Linking issues with K4FWcore/EDM4hep
* https://github.com/key4hep/K4FWCore/issues/9


### Nightly Builds and CI
- can we organize nightly builds of PODIO to be used for EDM4hep CI ?
    - should be addressed with spack releases mentioned above
- can we add a Ubuntu build to the CI ?
    - have observed build (link) issues on Ubuntu
        - "--as-needed" option added to calls to the linker
    - Ubuntu: 18.04 / 20.04 (LTS releases), system compiler gcc-7

## Podio


#### Change Relation Add Method Naming
* Done: https://github.com/AIDASoft/podio/pull/88

### Open Issues

- Run/Event/Collection meta data
    - https://github.com/AIDASoft/podio/issues/49
    - https://github.com/AIDASoft/podio/pull/71
    - Will evolve following tests of EDM4hep in FCCsw, for example
    - Store collection metadata once per file, as it should not change
    - Event metadata
        - Need to store different weights, pile up profile,
        - need survey of hh requirements

- Schema Evolution still an open issue
 
#### Schema Evolution

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


#### Meta Data

- Work in progress based on LCIO LCParameters (GenericParameters)
- Storing and reading them on demand is tricky, need table of contents to locate correct instance of parameters
- Just do it and make IO performent later?
- Also changes in Readers/Writers/EventStore necessary

#### EventStore

- Needs some reworking and definition of interfaces


## LCIOConverters

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

### Writing second file with another tree
* Cannot write second file?
* Have to do: ``` TFile::Open("tmp.root", "RECREATE")``` ?
    * Doesn't help

## EDM4hep

### vector member names should be plural
* Remove the weights from clusters
* https://github.com/key4hep/EDM4hep/issues/55
* https://github.com/key4hep/EDM4hep/pull/58

### Associations
* DONE
- need weight
    - Are there in LCIO LCRelation class
    - Missing in Association classes
- Should be one to many?
- Generic Association collection?
    - See what was done for pLCIO

### Release 1.0

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


## AOB

With regards to Snowmass process
* Current status of Key4hep project
    * Work in progress, not ready for production use
    * Experimental use welcome
    * Delphes plugin will come soon
* For Snowmass, one should use existing iLCSoft (ILD, CLICdp), FCCSW, CEPCsw where applicable


### Next meeting:

    June 9, 2020

 

 

There are minutes attached to this event. Show them.