Skip to content

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

Open
hotdankmatter wants to merge 1 commit intolesgourg:masterfrom
hotdankmatter:master
Open

Code implementing the new ncdm-approximation on top of the current CLASS version.#216
hotdankmatter wants to merge 1 commit intolesgourg:masterfrom
hotdankmatter:master

Conversation

@hotdankmatter
Copy link
Copy Markdown

@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