Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate to new AnaAlgorithms? #1421

Open
kratsg opened this issue Mar 13, 2020 · 4 comments
Open

Migrate to new AnaAlgorithms? #1421

kratsg opened this issue Mar 13, 2020 · 4 comments

Comments

@kratsg
Copy link
Contributor

kratsg commented Mar 13, 2020

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

@jbossios
Copy link
Contributor

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
Copy link
Contributor Author

kratsg commented Mar 13, 2020

@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
Copy link
Contributor

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

@kratsg
Copy link
Contributor Author

kratsg commented Mar 13, 2020

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...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants