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

In bkfit, copy out final parameters from propagated or updated state based on the state of propagate-to-PCA. #336

Merged
merged 1 commit into from
Aug 10, 2021

Conversation

osschar
Copy link
Collaborator

@osschar osschar commented Jul 26, 2021

No description provided.

@osschar
Copy link
Collaborator Author

osschar commented Jul 27, 2021

@mmasciov
Copy link
Collaborator

MTV results in TTbar + PU50: http://uaf-10.t2.ucsd.edu/~mmasciov/MkFit_devs/MTV_PR336/
Differences between head and PR #336 appear to be marginal.

@slava77
Copy link
Collaborator

slava77 commented Jul 30, 2021

Differences between head and PR #336 appear to be marginal.

I was not expecting a significant effect on average; this is more about getting the logic correct.

@mmasciov
Copy link
Collaborator

mmasciov commented Aug 9, 2021

If there are no objections, I'm going to click the 'merge' button on this PR.

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.

3 participants