EDM4hep Discussion

Tuesday 2 Jul 2024, 09:00 → 10:00 Europe/Zurich

Zoom

Live Notes

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

Date: July 2, 2024
Indico: https://indico.cern.ch/event/1433416/

Connected: Andre, Benedikt

Apologies: Thomas

## Introduction and General Points

### Upcoming workshops / conferences
https://github.com/orgs/key4hep/projects/4/views/1
    
## Progress and discussion

### podio v1.0 (and v1.0.1) has been released
* https://github.com/AIDASoft/podio/releases
    
## Podio
* https://github.com/AIDASoft/podio/pulls
* https://github.com/AIDASoft/podio/issues
* https://github.com/orgs/AIDASoft/projects/2/views/1

### Merged PRs
* Marking move from one component type to another one as impossible for schema evolution
    * https://github.com/AIDASoft/podio/pull/622
* Make the TTree and RNTuple based backends write the GenericParameters the same way
    * https://github.com/AIDASoft/podio/pull/625
* Document storage details
    * https://github.com/AIDASoft/podio/pull/629
* Fix: OneToOneRelations in cloned objects
    * https://github.com/AIDASoft/podio/pull/632
    * Missed in v1.0 -> v1.0.1 
* A few minor fixes and updates
    * https://github.com/AIDASoft/podio/pull/633
    * https://github.com/AIDASoft/podio/pull/634

### RDataSource for podio
* https://github.com/AIDASoft/podio/pull/593
* Possible to have a generice `RDataSource` for all generated EDMs
* Some comments to be addressed

### Make collection iterators fulfill `LegacyInputIterator` and `input_iterator` concept (wip)
* https://github.com/AIDASoft/podio/pull/626
* Documentation + usability upgrade for algorithms

### `podio-dump` implementation in c++ (wip)
* https://github.com/AIDASoft/podio/pull/620
* Complaints here: https://github.com/key4hep/EDM4hep/issues/312
* ~10 times faster than python implementation
* Needs `fmt`-lib for formatting (or gcc > 13 / clang > 16)
* dumping of datamodels stored in files not easy to implement in c++
    * Would require to implement an in-memory conversion from JSON to YAML
    * Keep that functionality in python in new tool
        
### Frame serialization / deserialization
* https://github.com/AIDASoft/podio/issues/565
* Not part of v1.0
* Proposal from D. Lawrence: https://github.com/AIDASoft/podio/pull/579

### Leak in SIO Frame reading
* https://github.com/AIDASoft/podio/issues/594

## EDM4hep
* https://github.com/key4hep/EDM4hep/pulls
* https://github.com/key4hep/EDM4hep/issues
* https://github.com/orgs/key5hep/projects/5 

### Merged PRs
* Change probabilit to ndf in Vertex
    * https://github.com/key4hep/EDM4hep/pull/324
* Remove the `MCRecoTrackerHitPlaneAssociation`
    * https://github.com/key4hep/EDM4hep/pull/331
* Remove some of the drift chamber study types again
    * https://github.com/key4hep/EDM4hep/pull/333
* Documentation updates
    * https://github.com/key4hep/EDM4hep/pull/330 (remove parts of docstring not relevant to EDM4hep)
    * https://github.com/key4hep/EDM4hep/pull/335 (update of schema plot)
    
### Remove the dQ/dx related information from the Track
* https://github.com/key4hep/EDM4hep/pull/311
* Intra datamodel consistency for information
* [ ] Downstream PRs still need to be prepared

### Remove `radiusOfInnermostHit`
* https://github.com/key4hep/EDM4hep/pull/326
* [ ] How to fill information in conversion from EDM4hep -> LCIO
    * https://github.com/key4hep/k4EDM4hep2LcioConv/pull/72
    
### Make the Vertex have a proper `type` bitfield instead of a `primary` field
* https://github.com/key4hep/EDM4hep/pull/329
* [ ] Needs changes in FCCAnalyses
    * https://github.com/HEP-FCC/FCCAnalyses/pull/382
    * Attempted to make this as transparent as possible
    
### Overhaul the relations between Vertex and ReconstructedParticle
* https://github.com/key4hep/EDM4hep/pull/332
* [ ] Downstream PRs still need to be prepared

### Remove compatibility with podio < v1.0
* https://github.com/key4hep/EDM4hep/pull/334
* In principle already done with https://github.com/key4hep/EDM4hep/pull/327 in CMake. This just removes "unreachable" code

## Converter & MarlinWrapper

* Several PRs to keep nightly builds running in converter, MarlinWrapper and FWCore, see links in EDM4hep PRs
    
## AoB

## Next meeting:
* July 16, 09:00

09:00 → 09:05 Introduction

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

EDM4hep Github

Podio Github

09:50 → 09:51 Discussion

Speaker: Dr All