Skip to content

Jab updates#17

Open
jablazek wants to merge 8 commits intomasterfrom
JAB_updates
Open

Jab updates#17
jablazek wants to merge 8 commits intomasterfrom
JAB_updates

Conversation

@jablazek
Copy link
Copy Markdown
Contributor

This is mostly cosmetic, but I had two questions.

  1. Have I correctly treated GI and IG terms for IA? I have explicitly included both, but maybe CCL already does this bookkeeping?

  2. In PerturbationTheoryPk.ipynb, why doesn't the C_ell integrator crash? We set up the IA bias with a max of z=1. Then the lensing tracer goes to z=1.5. What is happening?

@jablazek
Copy link
Copy Markdown
Contributor Author

UPDATE: I've been playing with this a bit, and the agreement between the NLA output with the standard CCL method and the output using generalized p_k_a and the PTTracers gets pretty bad for a broad z distribution. This appears to be true even if the input z range for the generalized p_k_a covers the input nz range.

Ideas?

@nikosarcevic
Copy link
Copy Markdown
Contributor

hey @jablazek is this PR still relevant?

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.

2 participants