You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are many tasks that might require to access columns/histograms from multiple campaigns simultaneously.
Machine Learning
creating datacards
rebinning
plotting (e.g. combining 2022preEE and 2022postEE)
There is already the MLModelTrainingMixin [1] that allows us to train DNNs with multiple campaigns. However, this mixin cannot be reused for other types of MultiConfigTasks.
Instead we should build one central MultiConfigTask base task.
The MLModelTrainingMixin currently resolves Calibrators, Selector and Producers (CSPs) for each year individually. Therefore, we cannot rely on the already defined mixins for CSPs but have to reimplement the resolving of each of those.
This could be simplified by removing the resolving for each year individually, since users can simply write their CSPs such that they are year-independent. Such a Producer might look like this:
The main changes that are required in our current configs/tasks is that our mixins for CSPs + ML need to inherit from the AnalysisTask instead of the ConfigTask and that the default CSPs need to be defined in the analysis_inst instead of the config_inst
There are many tasks that might require to access columns/histograms from multiple campaigns simultaneously.
There is already the
MLModelTrainingMixin
[1] that allows us to train DNNs with multiple campaigns. However, this mixin cannot be reused for other types of MultiConfigTasks.Instead we should build one central
MultiConfigTask
base task.The
MLModelTrainingMixin
currently resolves Calibrators, Selector and Producers (CSPs) for each year individually. Therefore, we cannot rely on the already defined mixins for CSPs but have to reimplement the resolving of each of those.This could be simplified by removing the resolving for each year individually, since users can simply write their CSPs such that they are year-independent. Such a Producer might look like this:
The main changes that are required in our current configs/tasks is that our mixins for CSPs + ML need to inherit from the
AnalysisTask
instead of theConfigTask
and that the default CSPs need to be defined in theanalysis_inst
instead of theconfig_inst
@riga
[1]
columnflow/columnflow/tasks/framework/mixins.py
Line 1014 in e786a35
The text was updated successfully, but these errors were encountered: