Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing citations for AraTha and EscCol. #527

Merged
merged 1 commit into from May 25, 2020

Conversation

grahamgower
Copy link
Member

Closes #355.

Theses species were added by @fbaumdicker and @arundurvasula --- would you be kind enough to quickly check that I haven't messed anything up here? Note that for AraTha, the recombination rate was listed as 8.1e-9, but I get 200 / 124000 / 2 / 1e6 = 8.064516129032258e-10.

@jeromekelleher
Copy link
Member

LGTM

@codecov
Copy link

codecov bot commented May 12, 2020

Codecov Report

Merging #527 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #527   +/-   ##
=======================================
  Coverage   99.68%   99.68%           
=======================================
  Files          21       21           
  Lines        1902     1904    +2     
  Branches      202      202           
=======================================
+ Hits         1896     1898    +2     
  Misses          2        2           
  Partials        4        4           
Impacted Files Coverage Δ
stdpopsim/catalog/AraTha.py 100.00% <ø> (ø)
stdpopsim/catalog/EscCol.py 100.00% <100.00%> (ø)

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 e0682fb...ec7fa83. Read the comment docs.

Copy link
Contributor

@fbaumdicker fbaumdicker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jeromekelleher
Copy link
Member

I think this is ready to merge after a rebase.

@grahamgower
Copy link
Member Author

Ok, merging. Thanks @jeromekelleher and @fbaumdicker!

@grahamgower grahamgower merged commit 86a74fa into popsim-consortium:master May 25, 2020
@grahamgower grahamgower deleted the cite branch May 26, 2020 07:20
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.

mutation_rate_citations needed for arabidopsis and e_coli.
3 participants