Skip to content

Refactoring of 2pc code#45

Open
Marcellocosti wants to merge 2 commits into
flowHF:devfrom
Marcellocosti:2pc
Open

Refactoring of 2pc code#45
Marcellocosti wants to merge 2 commits into
flowHF:devfrom
Marcellocosti:2pc

Conversation

@Marcellocosti

Copy link
Copy Markdown
Contributor

@wuctlby

wuctlby commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Hi @Marcellocosti , thanks a lot for the work! I believe you have already tested it locally, so it looks good to me. Just one minor comment,
for correlations/PostProcessing/FitCorrel.py to correlations/PostProcessing/fit_correl.py
and correlations/PostProcessing/construct_v2_mass.py to correlations/PostProcessing/extract_ry_triggers.py.
Thanks for the refactoring. I would suggest renaming them rather than deleting the original file, then creating a new one. It's hard to trace what we did in the past and validate the underlying logic

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

Successfully merging this pull request may close these issues.

2 participants