Skip to content

Add more utility functions for setting collection level parameters#391

Open
tmadlener wants to merge 7 commits intomainfrom
cellid-encoding-helpers
Open

Add more utility functions for setting collection level parameters#391
tmadlener wants to merge 7 commits intomainfrom
cellid-encoding-helpers

Conversation

@tmadlener
Copy link
Copy Markdown
Member

@tmadlener tmadlener commented Apr 1, 2026

BEGINRELEASENOTES

  • Add k4FWCore::putCollectionParameter and k4FWCore::getCollectionParameter to facilitate setting and getting collection level parameters / metadata
  • Add k4FWCore::putCellIDEncoding and k4FWCore::getCellIDEncoding to facilitate setting and getting CellID encoding strings
  • Adapt some tests to use these instead of MetaDataHandles

ENDRELEASENOTES

These simply wrap (get|put)Parameter, but call podio::collMetadataParamName (with edm4hep::labels::CellIDEncoding) under the hood first before passing things on. This is a follow up on this discussion.

@jmcarcell it looks like currently the only test for CellID encoding goes through a non-functional algorithm. I think we should add one for functional algorithms as well. Do you have a preference on whether

  • I should add a completely new Functional algorithm for this, or
  • I should re-use an existing one (e.g. the ExampleFunctionalProducerMultiple)?

@tmadlener tmadlener force-pushed the cellid-encoding-helpers branch from da8300a to d2da1b4 Compare April 1, 2026 12:28
@tmadlener tmadlener changed the title Add utility functions to get and set CellID encodings more easily Add more utility functions for setting collection level parameters Apr 2, 2026
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