EDM4hep Discussion
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