Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Transparently switch to optional parameter values #59

Closed
wants to merge 1 commit into from

Conversation

tmadlener
Copy link

BEGINRELEASENOTES

  • Introduce pre-processor checks to transparently switch to the new std::optional return values of podio::Frame::getParameter (introduced with AIDASoft/podio#580)

ENDRELEASENOTES

@tmadlener tmadlener force-pushed the generic-param-optional branch from 24fa7e4 to 81a58a7 Compare May 28, 2024 12:00
@tmadlener
Copy link
Author

Tests are currently failing due to incompatible input file: https://github.com/key4hep/k4Clue/actions/runs/9268881385/job/25498496792?pr=59#step:3:344

@tmadlener
Copy link
Author

This becomes obsolete with key4hep/k4FWCore#195

@tmadlener tmadlener closed this May 31, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant