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

Handle common code between training and inference pipelines #129

Open
ekauffma opened this issue May 1, 2023 · 1 comment
Open

Handle common code between training and inference pipelines #129

ekauffma opened this issue May 1, 2023 · 1 comment
Assignees
Labels
implementation concerns analysis implementation

Comments

@ekauffma
Copy link
Collaborator

ekauffma commented May 1, 2023

Currently, the training and inference pipelines in the CMS open data ttbar notebook have overlapping code which is simply duplicated at this point. This will be difficult to maintain, so it would be good to import these common features from one file.

Commonalities:

  • Calculation of top mass using trijet reconstruction method
  • Cuts
  • Options for processing
@alexander-held alexander-held added the implementation concerns analysis implementation label May 1, 2023
@alexander-held
Copy link
Member

Some of the visualization code in the training notebook also looks quite similar, perhaps there it is possible to factor out functionality to utils as well and to re-use it for plotting.

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

No branches or pull requests

2 participants