joshwlambert / DAISIEprep

Extracts phylogenetic island community data from phylogenetic trees
https://joshwlambert.github.io/DAISIEprep
GNU General Public License v3.0
6 stars 3 forks source link

Draft ASR vignette introduction section #4

Closed TheoPannetier closed 2 years ago

TheoPannetier commented 2 years ago

This section should explain the reader why it is relevant to perform ASR for DAISIE data preparation. It should also describe an overview of the examples that are going to be covered through the vignette.

TheoPannetier commented 2 years ago

The whole exercise might be a bit more convoluted than I anticipated; the assumptions of DAISIE do not necessarily match the processes modelled by DEC and SSE models. It would be good to remind of this in the intro (or a concluding section?). Developping a DAISIE-based ASR routine would probably be an appropriate solution but it's a challenging problem. For now, I'll focus on making the simple examples with DEC / SSE ASRs and come write the introduction later.

joshwlambert commented 2 years ago

@TheoPannetier this is not really a problem, the assumptions of DAISIE are used in a different algorithm in DAISIEprep by setting extraction_method = "min" in extract_island_species(). It is a good point you make that the differences between the assumptions of DAISIE and the ASR method of choice should be realised and explained in the vignette.

TheoPannetier commented 2 years ago

Oh right, I hadn't realised that "min" was doing just that. Brilliant then, if anything I'm more aware now of the conceptual differences between DAISIE and SSE/DEC models.

TheoPannetier commented 2 years ago

TODO: discuss why it's not that simple to do ASR with DAISIE, show an example of a tricky tree (NE clade nested within E clade) where the "min" algorithm returns counterintuitive results.

TheoPannetier commented 2 years ago

Finished intro draft, ready for review / PR