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

PodioOutput: add ability to promote subset to full collections #226

Open
Zehvogel opened this issue Aug 22, 2024 · 2 comments
Open

PodioOutput: add ability to promote subset to full collections #226

Zehvogel opened this issue Aug 22, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@Zehvogel
Copy link
Contributor

Sometimes we would like to drop the original collection and only write out one of its subset collections, for this we would need to "promote" the former subset collection to be a real collection. We could either try to automatically determine when this happens or have the user supply a dedicated list of subset collections that should be promoted as in Marlin/LCIO.

@Zehvogel Zehvogel added the enhancement New feature or request label Aug 22, 2024
@tmadlener
Copy link
Contributor

This will most likely require a bit of work also in podio because

  • subset collections are currently effectively stored in the same way as relations, i.e. we only store the object id for each element in a subset collection and then resolve this again when reading back. This simply needs a "bit of work" when storing a subset collection, to instead store itself as if it were a proper collection, mostly by simply cloning the necessary elements.
  • However, there is also a conceptual problem here, because a subset collection is effectively a collection of handles. Hence, if some object has a relation to an element of a subset collection, that relation is not aware that there is a subset collection in play and it will only know about the "original" object in the full collection that the subset collection points into. I am not sure if there is a general way of handling this.

@Zehvogel
Copy link
Contributor Author

This will most likely require a bit of work also in podio

I feared that this will be the case, I also opened a ticket there now :)

AIDASoft/podio#655

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants