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

Use templated getMap access for podio GenericParameters #1112

Merged
merged 3 commits into from
Jun 9, 2023

Conversation

tmadlener
Copy link
Contributor

BEGINRELEASENOTES

  • Use the templated getMap access functionality for podio::GenericParameters as the non-templated versions will be removed.

ENDRELEASENOTES

@github-actions
Copy link

github-actions bot commented May 19, 2023

Test Results

       6 files         6 suites   5h 28m 14s ⏱️
   345 tests    345 ✔️ 0 💤 0
1 025 runs  1 025 ✔️ 0 💤 0

Results for commit 434c0d9.

♻️ This comment has been updated with latest results.

parameters.data->stringParams = inputparams.getStringMap();
parameters.data->floatParams = inputparams.getFloatMap();
parameters.data->intParams = inputparams.getIntMap();
parameters.data->stringParams = inputparams.getMap<std::string>();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add #include <string>?

@MarkusFrankATcernch MarkusFrankATcernch merged commit 6eaddf5 into AIDASoft:master Jun 9, 2023
# 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.

3 participants