-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
podio::Frame
based I/O
#94
Comments
@tmadlener - at the moment I am stuck at https://github.com/key4hep/k4FWCore/blob/master/k4FWCore/include/k4FWCore/DataHandle.h#L82 |
I think as far as Key4hep is concerned, this is not a use case that we support any longer. This should be handled by the |
great. then I will just kill it for good |
Seems we actually have our very basic tests all depend on it :-( |
Discussed in issue #69 |
Done with #100 |
Since the currently used
EventStore
based I/O functionality is de-facto deprecated and will be removed from podio, we should switch to the Frame based I/O model. Ideally we only use podio components here and do not roll a separate implementation of Readers/Writers. Another goal is to support parallelism of any kind (inter-event and intra-event) out of the box, so that we can also easily switch all algorithm developments to the Gaudi Functional approach. To make this work we have to fulfill the correct Gaudi interface. We (@vvolkl and me) had a brief meeting with Marco Clemencic from the Gaudi team trying to identify the correct one. I will try to put the summary of that meeting here, although it will potentially not bring us all the way to the end, and we might have to have another meeting. (Obviously, the possibility exists that I misunderstood a few things)EventContext
AlgTask
(GaudiHive) is managing this context switching and also callssysExecute
on the managed algorithms to actually run themIDataProviderSvc
seems to be the interface that we need to fulfill. The basic idea would be to make each instance of this service own exactly onepodio::Frame
and leave the rest of the management to GaudiThe text was updated successfully, but these errors were encountered: