Skip to content

DPL Analysis: tutorial task demonstrating configurable-wrapped class#4537

Merged
jgrosseo merged 5 commits into
AliceO2Group:devfrom
aalkin:callable-configurable
Oct 14, 2020
Merged

DPL Analysis: tutorial task demonstrating configurable-wrapped class#4537
jgrosseo merged 5 commits into
AliceO2Group:devfrom
aalkin:callable-configurable

Conversation

@aalkin
Copy link
Copy Markdown
Member

@aalkin aalkin commented Oct 8, 2020

Adds an example demonstrating how to use configurable-wrapped class (e.g. for a cut object) in a task, with a sample of JSON configuration.

  • MutableConfigurable<> variant to be able to change configurable state in the task
  • configurableCut class with linkdef definition
  • configurableObjects task using the class both in const and mutable forms

@aalkin aalkin requested review from a team, iarsene and jgrosseo as code owners October 8, 2020 09:32
@jgrosseo
Copy link
Copy Markdown
Collaborator

jgrosseo commented Oct 8, 2020

Hi. Thanks for this. Before merging, we should discuss how we consolidate this with the idea of having a 2d parameter store with labels on "both axis". And how we represent all cases in the produced json, so that hyperloop can read them.

@aalkin
Copy link
Copy Markdown
Member Author

aalkin commented Oct 14, 2020

@jgrosseo @ktf this is already usable in its current state and will not interfere with later development of "2D" store, can we merge this?

@jgrosseo
Copy link
Copy Markdown
Collaborator

At the moment it will not work on hyperloop, but apart from that I have no objections.

@jgrosseo jgrosseo merged commit aaf8bd3 into AliceO2Group:dev Oct 14, 2020
@aalkin aalkin deleted the callable-configurable branch December 16, 2020 09:19
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