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
43 stars 126 forks source link

Migrate to new AnaAlgorithms? #1421

Open kratsg opened 4 years ago

kratsg commented 4 years ago

https://atlassoftwaredocs.web.cern.ch/AnalysisTools/ana_alg_migrate_el/

jbossios commented 4 years ago

For me, it would beneficial to be able to run on Athena. Although, I guess this will mean all xAH-based algorithms not living on xAH will also need to be updated.

kratsg commented 4 years ago

@jbossios xAH can configure algorithms of both old/new styles equivalently. This is just converting the algorithms inside xAH to AnaAlgorithm. Not going to affect other people. And nominally should see 0 difference.

jbossios commented 4 years ago

Then it would be awesome to do the migration. I hope we hear from others benefiting from this.

kratsg commented 4 years ago

The only thing I might need some help with is to figure out the re-ordering of some of the algorithms. I know there are subtle changes that break existing logic in our code because we assume initialize/fileExecute are in some order -- but I know one of those options gets dropped... but...