Bolt.jl icon indicating copy to clipboard operation
Bolt.jl copied to clipboard

Place to add (and test) quick features

Open jmsull opened this issue 2 years ago • 7 comments

jmsull avatar Jun 07 '22 07:06 jmsull

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.

jmsull avatar Jun 07 '22 07:06 jmsull

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.

codecov-commenter avatar Jun 07 '22 07:06 codecov-commenter

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 avatar Sep 23 '22 21:09 jmsull

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 avatar Oct 03 '22 17:10 jmsull

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

jmsull avatar Nov 03 '22 06:11 jmsull

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 avatar Nov 10 '22 23:11 jmsull

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.

jmsull avatar Jan 19 '23 23:01 jmsull