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

Code implementing the new ncdm-approximation on top of the current CLASS version. #216

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hotdankmatter
Copy link

@hotdankmatter hotdankmatter commented May 19, 2018

Hi Julien,

as the commit implied, implemented all parts necessary for using the AHL-fluid approximation in synchronous gauge.
This comprises:

  1. Added the necessary code in perturb_vector_init(), perturb_derives(), and perturb_total_stress_energy()
  2. Added ncdmfa_ahl to the possible values of ppr->ncdm_fluid_approximation as the 3rd value, no approximation means now 4.
  3. Intercepts the case when trying to use the AHL-approximation outside synchronous gauge
  4. Due to point 3, the fluid approximation is retained as standard method, but with the more beneficial value of ktau_switch for NCDM.
  5. Fixed a small bug where the could could complain about "switching 0 approximation schemes at the same time"

This code has so far been tested to work under most regular conditions, and the output test notebook works as expected.

Cheers and let me know if there are any more questions.

… in synchronous gauge.

1) Added the necessary code in perturb_vector_init(), perturb_derives(), and  perturb_total_stress_energy()
2) Added ncdmfa_ahl to the possible values of ppr->ncdm_fluid_approximation as the 3rd value, no approximation means now 4.
3) Intercepts the case when trying to use the AHL-approximation outside synchronous gauge
4) Due to point 3, the fluid approximation is retained as standard method, but with the more value of ktau_switch
5) Fixed a small bug where the could could complain about "switching 0 approximation schemes at the same time"
This code has so far been tested to work under most regular conditions.
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