-
Notifications
You must be signed in to change notification settings - Fork 87
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
Support internal samples in simulation through pedigree #1855
Comments
Ran into the error pointing me to this issue. I am interested in this as well. |
I am also interested, as I would like some ground-truth examples of simulations with non-contemporaneous samples and known pedigrees to test tsinfer (see tskit-dev/tsinfer#602). |
I'm thinking about this a bit more. Based on some recent discussion in the msprime Slack channel, I realized that the DTWF model also doesn't allow ancient samples to be direct ancestors of more recent samples, so it's not possible to sample direct (grand)parent-offspring relationships under DTWF. It would be great to be able to do this within the pedigree (e.g. simulate a bunch of trios within a population). Would there need to be serious reworking of the backend, or is it mostly a matter of testing to make sure labeling ancient individuals as samples within the pedigree is doing what we expect? From #1846, it was hard to tell what the sticking point is here. |
There's a note on it here. In principle it's simple - we just have to replace the existing segment chain for the sample
(excuse the mixed up pseudocode!) Hmm, that seems pretty straightforward actually - fancy having a go at it on the Python algoriths.py version @apragsdale ? |
Yes, I’d be happy to!
On 24 Nov 2021, at 13:33, Jerome Kelleher ***@***.***> wrote:
There's a note on it here<https://github.com/tskit-dev/msprime/blob/7fbaab3b031f7208cd82b79763e924260f412723/lib/msprime.c#L2184>.
In principle it's simple - we just have to replace the existing segment chain for the sample a with a new chain (0, L, a) (so that we have ancestry everywhere for our sample) and insert edges for every segment x in the old chain, like
for (x = existing_segments_for_a_head; x != NULL; x = x->next) {
add_edge(x.left, x.right, a, x.node)
}
(excuse the mixed up pseudocode!)
Hmm, that seems pretty straightforward actually - fancy having a go at it on the Python algoriths.py version @apragsdale<https://github.com/apragsdale> ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#1855 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AIBL46XJUA42WBSQOT62GC3UNU4XHANCNFSM5FLX6ERA>.
Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
This functionality to sample subjects internal to the pedigree would be very useful to research I am conducting on probabilities of rare variant sharing by related individuals. I hope this is implemented soon in msprime. |
Thanks for letting us know you're interested in this feature @abureau. There's no immediate plans to work on it unfortunately, unless anyone wants help out by doing some coding/testing. |
I would be happy to test the implementation, but I do not master msprime to the point to code a new feature. |
Thanks @abureau - we'll definitely call on your help if/when it gets implemented. Unfortunately I can't dedicate any of my own time here, but I'm happy to help if someone else wants to do a bit of model coding. |
I just ran into the same error message pointing me to this issue. It would be really useful if this feature was implemented. |
I think the status is the same @diegovelizo - nobody has implemented it yet, unfortunately. I think we need someone who is motivated to code this up in the Python prototyper (algorithms.py) and test it a bit. The actual C implementation would be quite easy then. |
I secured funding from NSERC that I can use to pay a developer of msprime interested in implementing sampling of subjects internal to the pedigree. That person needs to be based in Canada so I can pay him or her from my NSERC funds. Anyone in Canada interested, please contact me. |
Wow, congratulations! Great news! |
That's great news @abureau! Would you like to have a chat soon to discuss? |
Thanks for offering to chat about this implementation @jeromekelleher . That will be useful once I have identified the programmer who will do the job. I will contact you then. |
Sounds good to me @abureau, good luck! |
I know it's unusual to contact someone in a Github issue, but I emailed you last week @jeromekelleher about resolving this issue and I want to make sure my emails were not blocked by your spam filter. If you need more time to reply, no problem. |
I just merged #2321 which I think probably solves the issue, but with minimal testing. What we need to do in order to close this issue is improve testing. There's three broad classes of test here, which I'll go through separately. C Unit tests.These are to make sure that we don't make an C level mistakes, like accessing uninitialised memory etc. There is basic C testing in #2321, but I'd like to see an additional Python Unit testsThese are the most important tests, and should be thorough. Testing style has evolved over time here, but I'm seeing the value of simple examples recently. I'd like to see some worked out examples of simulating through a pedigree with internal samples in a variety of cases, mainly (1) all nodes are samples; (2) we only have internal samples (not sure what'll happen there, actually; should probably add this to the C tests too); one of the nodes in an internal individual is a sample and the other isn't (not sure what happens there either); we have an internal sample in a disconnected bit of the pedigree. That's all I can think of right now, but thinking through all the awkward corner cases here is the real work. Statistical tests.These are done in verification.py, and are run manually.
Comparisons with DTWF seem to be our main statistical validation here, so hopefully that'll work. I think @ThierryAM and @PandaGab are keen to get stuck in with doing the coding here, but are new to the codebase (and area). Others interested who might like to help could suggest some example code or ways to test for the the properties we expect of trees with internal pedigree samples. |
@jeromekelleher I am keen to start with some Python tests. Can you please point me to a file and ideally line in the code base where I can see current tests for this functionality? I am scratching my head how to mark nodes as samples in this setting. In #2321 (comment) I marked an individual as a sample when importing pedigree which then makes both of individual’s nodes samples. |
The test_pedigree.py is where most of the tests would go. The most basic think would probably to add some more cases to TestSimulateThroughPedigree with internal samples, so at least we're running the code. In the simple test I did for the algorithms.py prototype here I marked a specific slice of the nodes as samples. This would be a good start anyway! |
A simplification made in #1846 was to disallow samples that are internal nodes in the pedigree (i.e., samples cannot have children). It would not be difficult to implement, it just needs some thinking through exactly what the semantics are.
The text was updated successfully, but these errors were encountered: