Skip to content
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

Broken associations when using edm4hep input #113

Closed
Zehvogel opened this issue May 3, 2023 · 6 comments · Fixed by #142 or #147
Closed

Broken associations when using edm4hep input #113

Zehvogel opened this issue May 3, 2023 · 6 comments · Fixed by #142 or #147

Comments

@Zehvogel
Copy link
Contributor

Zehvogel commented May 3, 2023

For details see key4hep/k4LCIOReader#31

Maybe this is not worth fixing and we should directly switch to key4hep/k4EDM4hep2LcioConv#11? What is the status on that?

@tmadlener
Copy link
Contributor

That one is effectively ready, in the sense that we don't expect any real API changes any longer, unless we find a serious bug. What would be necessary from the k4MarlinWrapper side is to re-implement the wrapping (and talking to Gaudi) using that. It should in principle be rather similar to the EDM4hep -> LCIO direction.

@Zehvogel
Copy link
Contributor Author

Zehvogel commented May 4, 2023

Is there a timeline for when the integration into k4marlinWrapper will be done?

@tmadlener
Copy link
Contributor

Not really yet. The main problem is having the time to implement the wrapping. Technically it should be rather straight forward. The earliest I can have a look is probably in two-ish weeks.

@Zehvogel
Copy link
Contributor Author

@tmadlener can you provide an update on this?

@tmadlener
Copy link
Contributor

Still no progress, unfortunately. I will start looking into this now.

@Zehvogel
Copy link
Contributor Author

@jmcarcell maybe also something to add to key4hep/key4hep-spack#502?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants