Skip to content

DPL: Allow overriding array values in ConfigParamRegistry - #15927

Open
aalkin wants to merge 3 commits into
AliceO2Group:devfrom
aalkin:add-configparamregistry-array-overrides
Open

aalkin wants to merge 3 commits into
AliceO2Group:devfrom
aalkin:add-configparamregistry-array-overrides

Conversation

@aalkin

@aalkin aalkin commented Oct 11, 2026 •

Copy link
Copy Markdown
Member

To handle aod-metadata- per origin without altering existing mechanisms in O2Physics, we need to override and remove what is stored in the device's ConfigParamRegistry. This adds methods to remove and replace corresponding subtrees in ptree.

@aalkin
aalkin requested a review from a team as a code owner October 11, 2026 07:20
@aalkin
aalkin force-pushed the add-configparamregistry-array-overrides branch 2 times, most recently from c26537c to 9918750 Compare October 14, 2026 07:35
* Protect from long default values in `--help` output by adding a size
  limit and manually adding reduced string representation
* Use pre-existing to/from JSON conversions to dump and retrieve
  labeled arrays in command line
* Allow passing only values if the labels do not change from defaults
* Test options retrieval for the driver and devices
* Add a code path that uses put_child rather than put for vectors that
  are stored as subtrees
* Add override tests
@aalkin
aalkin force-pushed the add-configparamregistry-array-overrides branch from 9918750 to 6c4bbb9 Compare October 27, 2026 19:41
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 6c4bbb9 at 2026-10-27 21:33:

No log files found

Full log here.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants