search

UCATLAS / xAODAnaHelpers

ATLAS Run 2 and Run 3 analysis framework for AnalysisTop and AnalysisBase for proton-smashing physics
https://ucatlas.github.io/xAODAnaHelpers/
Apache License 2.0
42 stars 127 forks source link

Unifying algorithm parameters #958

Open ntadej opened 7 years ago

ntadej commented 7 years ago

This is more like a "discussion" issue. Especially for systematics, there are lots of different namings for algorithm parameters for example:

I wonder how this should be unified and to be more descriptive towards new users of xAH. Unfortunately it will be hard to avoid non-backwards compatible changes ...

kratsg commented 7 years ago

Unification could be done using xAH::Algorithm but this is the nuclear option.

Backwards compatibility can be done by creating a list of old/new variable mappings and making sure that the code warns when the old one is being used and suggest the new one. We already do this for m_debug and m_verbose which needed to be migrated to work correctly in release 21.