EDM4hep Discussion

→ Europe/Zurich
Zoom

Zoom

Zoom Meeting ID
98484040528
Host
Andre Sailer
Useful links
Join via phone
Zoom URL

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

There are minutes attached to this event. Show them.
    • 09:00 → 09:05
      Introduction 5m
      Speakers: Andre Sailer (CERN), Frank-Dieter Gaede (Deutsches Elektronen-Synchrotron (DE)), Thomas Madlener (Deutsches Elektronen-Synchrotron (DESY))
    • 09:45 → 09:46
      Discussion 1m
      Speaker: Dr All