EDM4hep Discussion
Zoom
EDM4hep Live Notes
==================
Date: Sep 12, 2023
Indico: https://indico.cern.ch/event/1324597
Connected: Andre, Swathi, Erica, Tao, Juan, Thomas, Sanghyun, Juraj, Frank, Gerri, Joseph
Apologies: Benedikt
## Introduction and General Points
### Upcoming workshops / conferences
https://github.com/orgs/key4hep/projects/4/views/1
### Tutorial
- Need content
- Networking situtation at location
- Bandwith?
- squid proxy for cvmfs cache?
- complete container? Feasible on a months timeframe?
- probably needs last minute fixes
- container will be large (multiple GBs)
- Nightlies or stable release?
## Progress and discussion
### Frame integration into FW
* https://github.com/key4hep/k4FWCore/pull/100
* **Things are in a somewhat usable state again(?)**
### LCIO to EDM4hep conversion
* https://github.com/key4hep/k4MarlinWrapper/pull/114
* Everything switched over
* Started to work on shared mapping
* https://github.com/key4hep/k4EDM4hep2LcioConv/pull/21
* Should hopefully fix some more of the observed issues
* https://github.com/key4hep/k4MarlinWrapper/issues/113
### Schema evolution code generation
* Several different attempts over the summer
* https://github.com/AIDASoft/podio/pull/441
* https://github.com/AIDASoft/podio/pull/460
* https://github.com/key4hep/EDM4hep/pull/221
* Tried switching from genreflex to rootcling
* Hit a wall, because apparently nobody uses this @LHC
* Hit a bug in ROOT with original genreflex
* Got this to work with ROOT 6.28/06
* https://github.com/AIDASoft/podio/pull/472
* Schema evolution is working for one old schema version
* Cleanup and fixing "unrelated" tests still to do
* [ ] Need 6.28/06 in the stack / nightlies (?)
* Major hurdle: Renaming member variables
* ROOT makes it very hard to get to the original value
* [ ] Presentation with more details once merged
## Podio
* https://github.com/AIDASoft/podio/pulls
* https://github.com/AIDASoft/podio/issues
* https://github.com/orgs/AIDASoft/projects/2/views/1
### All PRs & issues of the summer
* https://github.com/issues?q=repo%3AAIDASoft%2Fpodio+archived%3Afalse+sort%3Aupdated-desc+updated%3A%3E2023-07-18
* Selection:
* reduce template instations from JSON output: https://github.com/AIDASoft/podio/pull/477
* member typedefs (e.g. `collection_type`, `value_type`, ...) in generated code: https://github.com/AIDASoft/podio/pull/465
* python bindings for Frame writing: https://github.com/AIDASoft/podio/pull/447
* units in datamodel definition file: https://github.com/AIDASoft/podio/pull/445
### RNTuple reading multiple files might not be working
* [ ] To be confirmed and put into an issue
## EDM4hep
* https://github.com/key4hep/EDM4hep/pulls
* https://github.com/key4hep/EDM4hep/issues
### Add a OneToMany relation from TrackerHit to RecIonizationCluster
* https://github.com/key4hep/EDM4hep/pull/211
* When in the reconstrcution chain is the `RecIonizationCluster` produced?
* Related do discussion in https://github.com/key4hep/EDM4hep/issues/217
* In CEPC track fitting and cluster counting are separated
* Can be/should be possible created at the same time as the `TrackerHit`
* [ ] Check how this would work conceptually
* Ties in to a general discussion on which TrackerHit types we want and how they should work together
* https://github.com/key4hep/EDM4hep/issues/218
### Content of `MCParticle`
* https://github.com/key4hep/EDM4hep/issues/208
* [ ] Talk to original implementors in LCIO
* Original reason still valid?
* [ ] Talk to HepMC3 authors
* Organize common meeting (after we have done our part)
* Mikael Bergren
* Juergen Reuter (Whizard)
* Andrew Buckley
* Moving from `float` to `double` should be straight forward and can be done
* Schema evolution necessary?
* [ ] Add to list of podio test cases
## AoB
## Next meeting:
* Sep 26, 09:00