Stdpopsim: QC for new Arabidopsis model (HuberTwoEpoch)

Created on 1 Dec 2019  路  10Comments  路  Source: popsim-consortium/stdpopsim

Started a PR for this new model here https://github.com/popgensims/stdpopsim/pull/225

Anyone up for QC? This should be quite straight forward as the model params are plainly listed in this supplement: https://static-content.springer.com/esm/art%3A10.1038%2Fs41467-018-05281-7/MediaObjects/41467_2018_5281_MOESM1_ESM.pdf

All 10 comments

I can handle this.

thanks @ndukler. take a look at table S1 for the two epoch model params. once we merge this we might as well do the three epoch model too

@andrewkern The model got merged in without rebasing. Could you quickly set master back one commit (807525af5626450a06553fea4283fdc2d0d10b67), then rebase the athal_huberModel then re-merge it?

To clarify, not the QC model, PR #225 .

sorry to be dense here-- where is the issue? looks like a single commit?

It is a single commit. It just wasn't rebased against the head of master before it was merged. I might be misreading it though. Thanks for looking!

Ok, I have the QC model ready.

so should I revert, rebase, and then merge for you?

Yeah, that should do it. Sorry for the inconvenience :(

Can we close this down @andrewkern

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Rosemeis picture Rosemeis  路  5Comments

grahamgower picture grahamgower  路  5Comments

gtsambos picture gtsambos  路  4Comments

jeromekelleher picture jeromekelleher  路  5Comments

David-Peede picture David-Peede  路  6Comments