RJCB: To quote JOSS guidelines: 'Begin your paper with a summary of the high-level functionality of your software for a non-specialist reader. Avoid jargon in this section' (emphasis mine). I feel there is too much jargon in this section, and here is my suggestion to reduce it

RJCB: I think the DG algorithm can be explained in less detail, to make room to show what the package does, e.g. by showing some code and resulting figures

No vignettes

After

remotes::install_github("joshwlambert/DAISIEmainland")

and checking the vignettes, I find there are no vignettes:

> vignette(package = "DAISIEmainland")
no vignettes found

I think a devtools::build_vignettes() for the package would fix this, but building vignettes -as far as I can remember- changed and was finicky.

Too much words

I quote from joss_guidelines.md:

The paper should be between 250-1000 words. Authors submitting papers significantly longer than 1000 words may be asked to reduce the length of their paper.

Here we have approximately 1330 words:

wc paper.md --words
1330 paper.md

I would prefer there to be no code in the paper other than where unavoidable. Instead I suggest we use vignettes in the R package to allow ourselves to give a much more detail exposition of the code.

RJCB: I feel the paper is too detailed in general. I suggest to show what the package does and explaining from there. I suggest to show these aspects, that I quote from the summary:]

(1) simulate the evolutionary history on islands, (2) visualise that history,
(3) calculate and plot summary metrics of the simulated data

• Use own branch
• Create PR to Josh