Skip to content

add function for computing GL FSE correction factors#254

Open
kostrzewa wants to merge 3 commits intomasterfrom
gl_fse
Open

add function for computing GL FSE correction factors#254
kostrzewa wants to merge 3 commits intomasterfrom
gl_fse

Conversation

@kostrzewa
Copy link
Member

No description provided.

@kostrzewa kostrzewa requested review from pittlerf and urbach February 20, 2020 20:12
@kostrzewa
Copy link
Member Author

I guess the documentation needs some formatting love...

@kostrzewa
Copy link
Member Author

Since I went with just returning the correction factors, K_fps and K_Mps in a data frame, rather than the infinite-volume values of M_ps and f_ps, I've adjusted the documentation and removed the now unnecessary parameters for the mass and decay constant.

Copy link
Contributor

@pittlerf pittlerf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Bartek,
Am I right that a factor of 2 is missing from the correction to M_ps ?
K_Mps = (1+ 1/(2N_f)xig1_tilde), where Nf=2
so
K_Mps = (1+ 0.25
xi*g1_tilde),

@kostrzewa
Copy link
Member Author

Hi Bartek,
Am I right that a factor of 2 is missing from the correction to M_ps ?
K_Mps = (1+ 1/(2_N_f)_xi_g1_tilde), where Nf=2 so K_Mps = (1+ 0.25_xi*g1_tilde),

We work in the normalisation where f_pi ~ 130.4 MeV and \xi = (M_pi / 4 \pi f_pi)^2, so the formulas should be correct as implemented for this convention. When F_pi = f_pi/sqrt(2) is used instead, the factors change of course.

@pittlerf
Copy link
Contributor

Hi Bartek,
Am I right that a factor of 2 is missing from the correction to M_ps ?
K_Mps = (1+ 1/(2_N_f)_xi_g1_tilde), where Nf=2 so K_Mps = (1+ 0.25_xi*g1_tilde),

We work in the normalisation where f_pi ~ 130.4 MeV and \xi = (M_pi / 4 \pi f_pi)^2, so the formulas should be correct as implemented for this convention. When F_pi = f_pi/sqrt(2) is used instead, the factors change of course.

Thanks, I have just realized also that.

@kostrzewa
Copy link
Member Author

I still need to fix the issues with the documentation, so don't merge this yet please...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants