EDM4hep Discussion
Zoom
EDM4hep Discussion
- Zoom Meeting ID
- 98484040528
- Host
- Andre Sailer
- Useful links
- Join via phone
- Zoom URL
-
EDM4hep Live Notes
==================
Date: August 27, 2024
Indico: https://indico.cern.ch/event/1445644/
Connected: Andre, Swathi, Tao, Juan, Mateusz, Jacopo, Leonhard, Brieuc, Pere,
Apologies: Thomas, Frank
## Introduction and General Points
### Upcoming workshops / conferences
https://github.com/orgs/key4hep/projects/4/views/1
## Progress and discussion
## Podio
* https://github.com/AIDASoft/podio/pulls
* https://github.com/AIDASoft/podio/issues
* https://github.com/orgs/AIDASoft/projects/2/views/1
### Merged PRs
### 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
### Templated links between arbitrary datatypes
* [ ] https://github.com/AIDASoft/podio/pull/257
### Make collection iterators fulfill `LegacyInputIterator` and `input_iterator` concept (wip)
* [ ] https://github.com/AIDASoft/podio/pull/626 (Draft)
* [ ] 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/key4hep/projects/5
### Merged PRs
* [Drop support for the ROOTLegacyReader when converting to JSON](https://github.com/key4hep/EDM4hep/pull/342)
* [Remove a few unnecessary includes](https://github.com/key4hep/EDM4hep/pull/351)
* [Make associations links and prepare them for a potential introduction of templated links](https://github.com/key4hep/EDM4hep/pull/341)
* [Update the names of the links in the diagram](https://github.com/key4hep/EDM4hep/pull/349)
### Handling of links / associations
* See also discussion for naming in https://github.com/key4hep/EDM4hep/issues/344
* [x] Changes in https://github.com/key4hep/EDM4hep/pull/341
* ExtraCode to keep (almost) transparent for downstream
* k4SimGeant4 has some forward declarations that need fixing
* Current implementation is very similar to what templated links do
* Possible to transparently "schema evolve" this
* PR in podio for templated links https://github.com/AIDASoft/podio/pull/257
* [x] Fix downstream breakage with PRs
* [x] Merge
* [ ] Let one nightly build pass and make pre-release tag
## Converter & MarlinWrapper
## AoB
* Make a release with EDM4hep pre-release tag
### New Key4hep Release
* let FCC (Brieuc) know before, they have to tag some things
* Tag DD4hep, understand regexSD first
* Fix CED Event Display transparency: https://github.com/key4hep/key4hep-spack/issues/625
## Next meeting:
* Sep 10, 09:00