Skip to content
Snippets Groups Projects

add a addPrivateToolInArray function to DualUseConfig.py

Merged Nils Erik Krumnack requested to merge krumnack/athena:tool_array_python into master

Unfortunately I don't have a really good way to test if it works, but the underlying C++ is tested, and this is hopefully a close enough duplication of what's happening for non-array tools that this is good enough.

Originally I tried to do something more involved, but that became sort of tedious given the logic of the python class. Maybe at some point I will come back and restructure the python class somewhat. Hopefully this is still workable as is.

cc @delsart @khoo @wbalunas (who asked for dual-use ToolHandleArray support for the jet group)

cc @akraszna (who originally wrote the python AnaAlgorithmConfig)

Merge request reports

Pipeline #2511006 passed

Pipeline passed for d6b07dae on krumnack:tool_array_python

Approval is optional

Merged by Edward MoyseEdward Moyse 4 years ago (Apr 20, 2021 6:17pm UTC)

Merge details

  • Changes merged into with 78402347.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading