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

Primordial bug fix #514

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Jobilthomas
Copy link

This is a bug fix for 'int primordial_inflation_find_phi_pivot' from primordial.c.

Errors

In the case epsilon<1 that starts from line 2853 target, aH_ratio_after_small_epsilon and a_ratio_after_small_epsilon have their value zero from their initialization.

phi_small_epsilon corresponding to small_epsilon(0.1) hasn't been found for the case epsilon<1.

Since we intend to find phi corresponding to small_epsilon, the 'if' condition of line 2576 has to be
'epsilon > ppr->primordial_inflation_small_epsilon' instead of 'epsilon > 1'.

Fixes

Since everything from line 2596 to 2849 is common for both 'epsilon>o.1' and 'epsilon<0.1', I took these parts out of 'if' condition.

Added code to find phi_left and phi_right for 'epsilon<0.1' (which is 'else' of the if condition) just after line 2593.

Removed everything from line 2853 to 3044 as the purpose of this part is already taken care of when we made line 2596 to 2849 common.

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.

1 participant