Update pirouette figure

richelbilderbeek / razzo

Research project by Giovanni Laudanno and Richel J.C. Bilderbeek

GNU General Public License v3.0

2 stars 2 forks source link

Closed richelbilderbeek closed 5 years ago

richelbilderbeek commented 5 years ago

This figure should be improved:

workflow

richelbilderbeek commented 5 years ago

These are the suggestions I remember from @Giappo:

workflow

Giappo commented 5 years ago

Things to update:

[x] Twin's clock and site model must be imported from G;
[x] Twin's tree prior is always BD?
[x] There are two different inference models that are picked: one for true, one for twin;

richelbilderbeek commented 5 years ago

Twin's tree prior is always BD?

Yes, indeed. Alternatively, we could have used Yule as well. Point is: it is a constant/fixed.

Giappo commented 5 years ago

Ok, this is the current state. Confirm whether it is complete or there is something missing

workflow4

Giappo commented 5 years ago

I'll adjust the main text to be in accordance with this figure. The changes will be in:

richelbilderbeek commented 5 years ago

It's AFAICS perfect. Good job :+1:

Giappo commented 5 years ago

I leave a proofread of the three modified sections (see below) to @richelbilderbeek . If he thinks that everything is ok, we can close this issue.

[x] Pipeline figure;

[x] Pipeline figure's caption;

[x] Twinning section;

richelbilderbeek commented 5 years ago

Yup, will proofread Monday before 17:00.

richelbilderbeek commented 5 years ago

Proofreading

richelbilderbeek commented 5 years ago

Done :+1:

richelbilderbeek commented 5 years ago

It's on the Overleaf as well :rainbow: