Skip to content

Restrict datahandle #319

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

Closed
wants to merge 6 commits into from
Closed

Conversation

tmadlener
Copy link
Contributor

BEGINRELEASENOTES

  • Restrict the DataHandle to work only for collections that are from podio generated EDMs
  • Remove unnecessary friendship with (deprecated and removed) Gaudi classes
  • Bump the minimum required version of podio to 1.3

ENDRELEASENOTES

Another small part that falls out of #318. It looks like there is no usage of the DataHandle throughout the stack that is not a podio collection, so we can simply require that at compile time to simplify the implementation where we no longer have to handle other types.

Have we decided on a convention yet on how to use concepts? I have for now gone with constraining them in the template specification directly, but we could also go via the the requires route and keeping the template<typename T> part.

@tmadlener
Copy link
Contributor Author

@tmadlener tmadlener closed this May 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant