xzackli / Bolt.jl

differentiable boltzmann code
MIT License
43 stars 5 forks source link

Place to add (and test) quick features #65

Closed jmsull closed 6 months ago

jmsull commented 2 years ago

I added the E polzn source function and some functions to compute clte, clee (which addesses #63) - all untested! Also addressed #62 by lazily copying and pasting the primordial power lines I used in the Plin function.

codecov-commenter commented 2 years ago

Codecov Report

Merging #65 (b5e4233) into main (f048e40) will decrease coverage by 4.51%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
- Coverage   79.41%   74.90%   -4.52%     
==========================================
  Files          11       11              
  Lines         996     1056      +60     
==========================================
  Hits          791      791              
- Misses        205      265      +60     
Impacted Files Coverage Δ
src/Bolt.jl 0.00% <ø> (ø)
src/perturbations.jl 79.26% <ø> (ø)
src/spectra.jl 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

jmsull commented 2 years ago

The class comparison test for perturbations failed because it is against a nonu run - will rename the test to be a nonu test where mnu is explicitly set to zero.

jmsull commented 2 years ago

Test is failing because for some unknown reason Interpolations.jl switched from camelcase "LinearInterpolation" to snake case "linear_interpolation". Test is using v0.13.6 - I was using v0.14.5 locally - will update the Manifest version.

jmsull commented 2 years ago

This is marked as updated in the manifest...not sure what is happening here...

jmsull commented 2 years ago

There is now a collision between "LinearInterpolations" in the 13 version of Interpolations.jl and the current version of DataInterpolations.jl. The solution is to update Interpolations.jl to 14 (which I tried to do above but failed)

jmsull commented 1 year ago

We need to merge this so that people can use the main branch - I will look at this again to try to make that happen - even if it means dropping DataInterpolations temporarily.