Orbital sampling

[elenagonzalez870]

Some binaries were being sampled at separations smaller than RL radius, causing them to merge immediately. To fix this, I added lines in the independent.py file that check if the orbital period sampled is less than the minimum orbital period for RL. If so, it replaces the value with the minimum RL separation. The same procedure is used for the sana12 and renzo19 samplers. For consistency, I also truncate the eccentricities to avoid RL overflow. For the sana12 eccentricity sampling, I set the maximum value to be either the sana12 value of 0.9 or the one corrected for RL, whichever is smaller.

[codecov]

Codecov Report

Merging #563 (448468c) into develop (428a66a) will decrease coverage by 0.85%.
The diff coverage is 87.50%.

@@             Coverage Diff             @@
##           develop     #563      +/-   ##
===========================================
- Coverage    88.25%   87.40%   -0.85%     
===========================================
  Files           40       40              
  Lines        25373    25392      +19     
===========================================
- Hits         22392    22193     -199     
- Misses        2981     3199     +218     

Impacted Files	Coverage Δ
cosmic/sample/sampler/independent.py	64.84% <87.50%> (+0.87%)	⬆️
cosmic/sample/sampler/multidim.py	17.57% <0.00%> (-68.92%)	⬇️
cosmic/evolve.py	84.58% <0.00%> (-5.00%)	⬇️
cosmic/sample/sampler/cmc.py	82.47% <0.00%> (ø)
cosmic/sample/initialcmctable.py	95.49% <0.00%> (+0.14%)	⬆️
cosmic/sample/initialbinarytable.py	94.44% <0.00%> (+0.16%)	⬆️

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

[katiebreivik]

Thanks @elenagonzalez870 !