COSMIC-PopSynth / COSMIC

COSMIC (Compact Object Synthesis and Monte Carlo Investigation Code)
GNU General Public License v3.0
46 stars 58 forks source link

fixed bug which supplied incorrect path to params file in cosmic-pop #523

Closed katiebreivik closed 3 years ago

katiebreivik commented 3 years ago

This is a great lesson in making sure that your non-automated tests are correct and testing the code you want them to!

Needed to supply args.inifile instead of args.params to the independent sampler in cosmic-pop

codecov[bot] commented 3 years ago

Codecov Report

Merging #523 (f9af630) into develop (f535eba) will increase coverage by 0.00%. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #523   +/-   ##
========================================
  Coverage    86.74%   86.74%           
========================================
  Files           40       40           
  Lines        25356    25362    +6     
========================================
+ Hits         21993    21999    +6     
  Misses        3363     3363           
Impacted Files Coverage Δ
cosmic/sample/initialcmctable.py 95.38% <0.00%> (+0.15%) :arrow_up:
cosmic/sample/initialbinarytable.py 94.44% <0.00%> (+0.16%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f535eba...f9af630. Read the comment docs.