[PRE REVIEW]: 4DModeller: a spatio-temporal modelling package #6996

editorialbot · 2024-07-16T11:31:52Z

Submitting author: @rwestaway (Richard Westaway)
Repository: https://github.com/4DModeller/fdmr
Branch with paper.md (empty if default branch): JOSS
Version: v0.2.0
Editor: @Nikoleta-v3
Reviewers: @PieterjanRobbe, @wcjochem
Managing EiC: Daniel S. Katz

Status

Status badge code:

HTML: <a href="https://joss.theoj.org/papers/0f2a730dc4786def16ae43c849e07a73"><img src="https://joss.theoj.org/papers/0f2a730dc4786def16ae43c849e07a73/status.svg"></a>
Markdown: [![status](https://joss.theoj.org/papers/0f2a730dc4786def16ae43c849e07a73/status.svg)](https://joss.theoj.org/papers/0f2a730dc4786def16ae43c849e07a73)

Author instructions

Thanks for submitting your paper to JOSS @rwestaway. Currently, there isn't a JOSS editor assigned to your paper.

@rwestaway if you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). You can search the list of people that have already agreed to review and may be suitable for this submission.

Editor instructions

The JOSS submission bot @editorialbot is here to help you find and assign reviewers and start the main review. To find out what @editorialbot can do for you type:

@editorialbot commands

The text was updated successfully, but these errors were encountered:

editorialbot · 2024-07-16T11:31:55Z

Hello human, I'm @editorialbot, a robot that can help you with some common editorial tasks.

For a list of things I can do to help you, just type:

@editorialbot commands

For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:

@editorialbot generate pdf

editorialbot · 2024-07-16T11:32:09Z

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1785/0220170246 is OK
- 10.1111/2041-210X.13168 is OK
- 10.3030/694188 is OK
- 10.1111/j.1467-9868.2008.00700.x is OK
- 10.2307/j.ctvjf9vz4 is OK
- 10.21105/joss.03914 is OK
- 10.25436/E27C7F is OK
- 10.1038/s41598-024-60964-0 is OK
- 10.1214/17-AOAS1078 is OK
- 10.1093/gji/ggac365 is OK

MISSING DOIs

- No DOI given, and none found for title: inlabru

INVALID DOIs

- None

editorialbot · 2024-07-16T11:32:12Z

Software report:

github.com/AlDanial/cloc v 1.90  T=0.08 s (746.5 files/s, 499974.9 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
CSV                              4              0              0          28871
Rmd                             17           1024           1364           1959
R                               21            289            532           1848
TeX                              2            110              0           1077
XML                              1              0             12            691
Markdown                         8            185              0            645
YAML                             4             11             12            210
JSON                             1              0              0              8
-------------------------------------------------------------------------------
SUM:                            58           1619           1920          35309
-------------------------------------------------------------------------------

Commit count by author:

   407	Gareth Jones
    41	Anrijs K. Abele
    40	XueqingYin
    18	mnky9800n
    12	John Aiken
     7	desireetreichler
     5	Anrijs Abele
     4	Tian Li
     3	Désirée Treichler
     2	Kristoffer Aalstad
     1	Alexander Minakov
     1	El
     1	Jonathan Bamber
     1	Julie Røste
     1	mmazzolini

editorialbot · 2024-07-16T11:32:16Z

Paper file info:

📄 Wordcount for paper.md is 989

✅ The paper includes a Statement of need section

editorialbot · 2024-07-16T11:32:18Z

License info:

🟡 License found: GNU General Public License v3.0 (Check here for OSI approval)

editorialbot · 2024-07-16T11:33:15Z

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

editorialbot · 2024-07-16T11:33:59Z

Five most similar historical JOSS papers:

nsink: An R package for flow path nitrogen removal estimation
Submitting author: @jhollist
Handling editor: @crvernon (Active)
Reviewers: @jmp75, @ldecicco-USGS, @jmp75
Similarity score: 0.6986

GaussianRandomFields.jl: A Julia package to generate and sample from Gaussian random fields
Submitting author: @PieterjanRobbe
Handling editor: @jbytecode (Active)
Reviewers: @ziyiyin97, @shahmoradi
Similarity score: 0.6979

WorldDynamics.jl: A Julia Package for Developing and Simulating Integrated Assessment Models
Submitting author: @aurorarossi
Handling editor: @fraukewiese (Active)
Reviewers: @ranocha, @StanczakDominik, @miguelraz
Similarity score: 0.6978

parafields: A generator for distributed, stationary Gaussian processes
Submitting author: @dokempf
Handling editor: @diehlpk (Active)
Reviewers: @shahmoradi, @gchure
Similarity score: 0.6949

dlmmc: Dynamical linear model regression for atmospheric time-series analysis
Submitting author: @justinalsing
Handling editor: @danielskatz (Active)
Reviewers: @Chilipp, @taqtiqa-mark
Similarity score: 0.6946

⚠️ Note to editors: If these papers look like they might be a good match, click through to the review issue for that paper and invite one or more of the authors before considering asking the reviewers of these papers to review again for JOSS.

danielskatz · 2024-07-16T11:37:38Z

👋 @Nikoleta-v3 - Here's a second submission that I wonder if you could edit? If you can take this one or the other, it would be great!

danielskatz · 2024-07-16T11:37:47Z

@editorialbot invite @Nikoleta-v3

editorialbot · 2024-07-16T11:37:49Z

I'm sorry human, I don't understand that. You can see what commands I support by typing:

@editorialbot commands

danielskatz · 2024-07-16T11:38:01Z

@editorialbot invite @Nikoleta-v3 as editor

editorialbot · 2024-07-16T11:38:03Z

Invitation to edit this submission sent!

rwestaway · 2024-07-16T16:23:10Z

Potential reviewers based on topic areas (none are known to us): Fernando de Pol Mayer, James Mba Azam, Nicholas Tierney, wcjochem, adithirgis, thomaspinder, stevenpawley

Nikoleta-v3 · 2024-07-19T04:37:03Z

👋 @Nikoleta-v3 - Here's a second submission that I wonder if you could edit? If you can take this one or the other, it would be great!

@danielskatz 👋🏻 I'm happy to review this submission 😄 Unfortunately, I can't do both at the moment.

Nikoleta-v3 · 2024-07-19T04:38:09Z

@editorialbot me as editor

editorialbot · 2024-07-19T04:38:11Z

I'm sorry human, I don't understand that. You can see what commands I support by typing:

@editorialbot commands

Nikoleta-v3 · 2024-07-19T04:38:29Z

@editorialbot commands

editorialbot · 2024-07-19T04:38:32Z

Hello @Nikoleta-v3, here are the things you can ask me to do:


# List all available commands
@editorialbot commands

# Add to this issue's reviewers list
@editorialbot add @username as reviewer

# Remove from this issue's reviewers list
@editorialbot remove @username from reviewers

# Get a list of all editors's GitHub handles
@editorialbot list editors

# Assign a user as the editor of this submission
@editorialbot assign @username as editor

# Remove the editor assigned to this submission
@editorialbot remove editor

# Remind an author, a reviewer or the editor to return to a review after a 
# certain period of time (supported units days and weeks)
@editorialbot remind @reviewer in 2 weeks

# Adds a checklist for the reviewer using this command
@editorialbot generate my checklist

# Set a value for version
@editorialbot set v1.0.0 as version

# Set a value for branch
@editorialbot set joss-paper as branch

# Set a value for repository
@editorialbot set https://github.com/organization/repo as repository

# Set a value for the archive DOI
@editorialbot set 10.5281/zenodo.6861996 as archive

# Mention the EiCs for the correct track
@editorialbot ping track-eic

# Run checks and provide information on the repository and the paper file
@editorialbot check repository

# Check the references of the paper for missing DOIs
@editorialbot check references

# Generates the pdf paper
@editorialbot generate pdf

# Recommends the submission for acceptance
@editorialbot recommend-accept

# Generates a LaTeX preprint file
@editorialbot generate preprint

# Flag submission with questionable scope
@editorialbot query scope

# Get a link to the complete list of reviewers
@editorialbot list reviewers

# Creates a post-review checklist with editor and authors tasks
@editorialbot create post-review checklist

# Open the review issue
@editorialbot start review

Nikoleta-v3 · 2024-07-19T04:38:50Z

@editorialbot assign me as editor

editorialbot · 2024-07-19T04:38:53Z

Assigned! @Nikoleta-v3 is now the editor

Nikoleta-v3 · 2024-07-20T17:27:13Z

Thank you for your submission and for suggesting reviewers @rwestaway. I have successfully managed to install the package and run a few examples. The package is very well documented, and the readme is very clear and well organized 😄

I am going to contact some of the authors of recent publications as well (see #6996 (comment)) to see if they can review the paper. In the meantime, could you please address some of my comments and questions?

Paper

There are some minor typos in the paper:

Line 27: "behaves"
Line 35: "postcode"
Line 72: "by the UK."

Also in the paper, you have two references for inlabru. Their Methods in Ecology and Evolution paper and the GitHub repository. Is there a reason for that?

Tests

I see that you have some tests. Could you include some information on how to run the test suite? I really like your contribution guidelines; they are short and to the point. Maybe adding the test information there? or would be a bit much?

Whatever you decide, it would just be useful to know how to run the tests locally since the tests run when people open pull requests.

Nikoleta-v3 · 2024-07-20T17:35:47Z

👋🏻 @jhollist @PieterjanRobbe @wcjochem @thomaspinder would any of you be willing to review this submission for JOSS?

We carry out our checklist-driven reviews here in GitHub issues and follow these guidelines: https://joss.readthedocs.io/en/latest/review_criteria.html

The submission I'd like you to review is titled: "4DModeller: a spatio-temporal modelling package". You can find more information at the top of this Github issue 🆙

Please let me know if you're available 😄 Thank you!

rwestaway · 2024-07-22T07:22:06Z

Thank you for your submission and for suggesting reviewers @rwestaway. I have successfully managed to install the package and run a few examples. The package is very well documented, and the readme is very clear and well organized 😄

I am going to contact some of the authors of recent publications as well (see #6996 (comment)) to see if they can review the paper. In the meantime, could you please address some of my comments and questions?

Paper

There are some minor typos in the paper:

Line 27: "behaves"

Line 35: "postcode"

Line 72: "by the UK."

Also in the paper, you have two references for inlabru. Their Methods in Ecology and Evolution paper and the GitHub repository. Is there a reason for that?

Tests

I see that you have some tests. Could you include some information on how to run the test suite? I really like your contribution guidelines; they are short and to the point. Maybe adding the test information there? or would be a bit much?

Whatever you decide, it would just be useful to know how to run the tests locally since the tests run when people open pull requests.

Thank you @Nikoleta-v3 for your prompt feedback. Should we make changes to the paper.md file as we go along or is that now 'frozen' until all reviewers have responded? We haven't submitted to JOSS before so are unsure of the process...

Paper

Typos will be corrected.

The distinction is between (1) the applications/tutorials of inlabru which are only available via the GitHub repository (cited as Lindgren et al., 2024) and (2) the underlying statistics/method for which the package authors provide two references at https://inlabru-org.github.io/inlabru/authors.html#citation (Bachl et al., 2019 and Yuan et al., 2017). Does that make sense or does it need further clarification in the text?

Tests

I will have to consult the project team on this one.

mnky9800n · 2024-07-22T11:50:18Z

Software report:

github.com/AlDanial/cloc v 1.90  T=0.08 s (746.5 files/s, 499974.9 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
CSV                              4              0              0          28871
Rmd                             17           1024           1364           1959
R                               21            289            532           1848
TeX                              2            110              0           1077
XML                              1              0             12            691
Markdown                         8            185              0            645
YAML                             4             11             12            210
JSON                             1              0              0              8
-------------------------------------------------------------------------------
SUM:                            58           1619           1920          35309
-------------------------------------------------------------------------------

Commit count by author:

   407	Gareth Jones
    41	Anrijs K. Abele
    40	XueqingYin
    18	mnky9800n
    12	John Aiken
     7	desireetreichler
     5	Anrijs Abele
     4	Tian Li
     3	Désirée Treichler
     2	Kristoffer Aalstad
     1	Alexander Minakov
     1	El
     1	Jonathan Bamber
     1	Julie Røste
     1	mmazzolini

FYI @mnky9800n is John Aiken and that is me. so don't know why it counts me separate but it s not a big deal i guess.

PieterjanRobbe · 2024-07-23T07:56:00Z

@Nikoleta-v3 I'm happy to help, but I do want to mention that I'm not an R expert.

Nikoleta-v3 · 2024-07-23T08:21:50Z

Thank you @Nikoleta-v3 for your prompt feedback. Should we make changes to the paper.md file as we go along or is that now 'frozen' until all reviewers have responded? We haven't submitted to JOSS before so are unsure of the process...

Either is fine! If you push to the JOSS branch, everything will be updated. To be honest, I used to wait until I found reviewers to give any feedback on the paper, but since finding reviewers can be quite challenging at times, I started giving feedback beforehand so there is not too much time wasted. This is new territory for me too, haha.

I think you are fine to make the changes, at least the paper changes, and push so the reviewers will have the latest version with the corrections.

Paper

Typos will be corrected.

The distinction is between (1) the applications/tutorials of inlabru which are only available via the GitHub repository (cited as Lindgren et al., 2024) and (2) the underlying statistics/method for which the package authors provide two references at https://inlabru-org.github.io/inlabru/authors.html#citation (Bachl et al., 2019 and Yuan et al., 2017). Does that make sense or does it need further clarification in the text?

Thank you, I understand; I believe it's fine as it is!

Tests

I will have to consult the project team on this one.

Great; thank you.

Nikoleta-v3 · 2024-07-23T08:24:20Z

@PieterjanRobbe Amazing, thank you! That is fine; between myself and the second reviewer, it should be fine!

Nikoleta-v3 · 2024-07-23T08:25:31Z

@editorialbot add @PieterjanRobbe as reviewer

editorialbot · 2024-07-23T08:25:34Z

@PieterjanRobbe added to the reviewers list!

rwestaway · 2024-07-24T09:42:31Z

@editorialbot generate pdf

editorialbot · 2024-07-24T09:43:56Z

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

editorialbot · 2024-07-24T09:44:39Z

Five most similar historical JOSS papers:

GaussianRandomFields.jl: A Julia package to generate and sample from Gaussian random fields
Submitting author: @PieterjanRobbe
Handling editor: @jbytecode (Active)
Reviewers: @ziyiyin97, @shahmoradi
Similarity score: 0.7018

WorldDynamics.jl: A Julia Package for Developing and Simulating Integrated Assessment Models
Submitting author: @aurorarossi
Handling editor: @fraukewiese (Active)
Reviewers: @ranocha, @StanczakDominik, @miguelraz
Similarity score: 0.6996

nsink: An R package for flow path nitrogen removal estimation
Submitting author: @jhollist
Handling editor: @crvernon (Active)
Reviewers: @jmp75, @ldecicco-USGS, @jmp75
Similarity score: 0.6990

dlmmc: Dynamical linear model regression for atmospheric time-series analysis
Submitting author: @justinalsing
Handling editor: @danielskatz (Active)
Reviewers: @Chilipp, @taqtiqa-mark
Similarity score: 0.6969

parafields: A generator for distributed, stationary Gaussian processes
Submitting author: @dokempf
Handling editor: @diehlpk (Active)
Reviewers: @shahmoradi, @gchure
Similarity score: 0.6962

⚠️ Note to editors: If these papers look like they might be a good match, click through to the review issue for that paper and invite one or more of the authors before considering asking the reviewers of these papers to review again for JOSS.

wcjochem · 2024-07-25T19:56:11Z

Please let me know if you're available 😄 Thank you!
@Nikoleta-v3 I'm happy to help with this, if you still need another reviewer

rwestaway · 2024-07-26T09:24:29Z

Tests

I will have to consult the project team on this one.

Great; thank you.

@Nikoleta-v3 - To respond to your query, the testing suite is a relic of some old code so thank you for flagging that! At the time we had some ideas for testing meshes, and other topics, but as it stands we haven't really built it out or developed it further. However it is something planned for future work.

editorialbot · 2024-07-26T09:26:02Z

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

editorialbot · 2024-07-26T09:26:48Z

Five most similar historical JOSS papers:

GaussianRandomFields.jl: A Julia package to generate and sample from Gaussian random fields
Submitting author: @PieterjanRobbe
Handling editor: @jbytecode (Active)
Reviewers: @ziyiyin97, @shahmoradi
Similarity score: 0.7019

WorldDynamics.jl: A Julia Package for Developing and Simulating Integrated Assessment Models
Submitting author: @aurorarossi
Handling editor: @fraukewiese (Active)
Reviewers: @ranocha, @StanczakDominik, @miguelraz
Similarity score: 0.6996

nsink: An R package for flow path nitrogen removal estimation
Submitting author: @jhollist
Handling editor: @crvernon (Active)
Reviewers: @jmp75, @ldecicco-USGS, @jmp75
Similarity score: 0.6988

dlmmc: Dynamical linear model regression for atmospheric time-series analysis
Submitting author: @justinalsing
Handling editor: @danielskatz (Active)
Reviewers: @Chilipp, @taqtiqa-mark
Similarity score: 0.6968

parafields: A generator for distributed, stationary Gaussian processes
Submitting author: @dokempf
Handling editor: @diehlpk (Active)
Reviewers: @shahmoradi, @gchure
Similarity score: 0.6962

⚠️ Note to editors: If these papers look like they might be a good match, click through to the review issue for that paper and invite one or more of the authors before considering asking the reviewers of these papers to review again for JOSS.

editorialbot · 2024-07-26T09:54:05Z

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

editorialbot · 2024-07-26T09:54:50Z

Five most similar historical JOSS papers:

GaussianRandomFields.jl: A Julia package to generate and sample from Gaussian random fields
Submitting author: @PieterjanRobbe
Handling editor: @jbytecode (Active)
Reviewers: @ziyiyin97, @shahmoradi
Similarity score: 0.7021

WorldDynamics.jl: A Julia Package for Developing and Simulating Integrated Assessment Models
Submitting author: @aurorarossi
Handling editor: @fraukewiese (Active)
Reviewers: @ranocha, @StanczakDominik, @miguelraz
Similarity score: 0.6999

nsink: An R package for flow path nitrogen removal estimation
Submitting author: @jhollist
Handling editor: @crvernon (Active)
Reviewers: @jmp75, @ldecicco-USGS, @jmp75
Similarity score: 0.6989

dlmmc: Dynamical linear model regression for atmospheric time-series analysis
Submitting author: @justinalsing
Handling editor: @danielskatz (Active)
Reviewers: @Chilipp, @taqtiqa-mark
Similarity score: 0.6970

parafields: A generator for distributed, stationary Gaussian processes
Submitting author: @dokempf
Handling editor: @diehlpk (Active)
Reviewers: @shahmoradi, @gchure
Similarity score: 0.6964

⚠️ Note to editors: If these papers look like they might be a good match, click through to the review issue for that paper and invite one or more of the authors before considering asking the reviewers of these papers to review again for JOSS.

Nikoleta-v3 · 2024-07-28T12:18:17Z

@wcjochem amazing thank you! 🙏🏻

Nikoleta-v3 · 2024-07-28T12:18:40Z

@editorialbot add @wcjochem as reviewer

editorialbot · 2024-07-28T12:18:42Z

@wcjochem added to the reviewers list!

Nikoleta-v3 · 2024-07-28T12:19:59Z

@editorialbot start review

editorialbot · 2024-07-28T12:20:02Z

OK, I've started the review over in #7047.

editorialbot added pre-review Track: 7 (CSISM) Computer science, Information Science, and Mathematics labels Jul 16, 2024

editorialbot added R TeX labels Jul 16, 2024

editorialbot assigned Nikoleta-v3 Jul 19, 2024

editorialbot assigned PieterjanRobbe Jul 23, 2024

editorialbot assigned wcjochem Jul 28, 2024

editorialbot closed this as completed Jul 28, 2024

[PRE REVIEW]: 4DModeller: a spatio-temporal modelling package #6996

[PRE REVIEW]: 4DModeller: a spatio-temporal modelling package #6996

Comments

editorialbot commented Jul 16, 2024 • edited Loading

Status

editorialbot commented Jul 16, 2024

editorialbot commented Jul 16, 2024

editorialbot commented Jul 16, 2024

editorialbot commented Jul 16, 2024

editorialbot commented Jul 16, 2024

editorialbot commented Jul 16, 2024

editorialbot commented Jul 16, 2024

danielskatz commented Jul 16, 2024

danielskatz commented Jul 16, 2024

editorialbot commented Jul 16, 2024

danielskatz commented Jul 16, 2024

editorialbot commented Jul 16, 2024

rwestaway commented Jul 16, 2024 • edited Loading

Nikoleta-v3 commented Jul 19, 2024

Nikoleta-v3 commented Jul 19, 2024

editorialbot commented Jul 19, 2024

Nikoleta-v3 commented Jul 19, 2024

editorialbot commented Jul 19, 2024

Nikoleta-v3 commented Jul 19, 2024

editorialbot commented Jul 19, 2024

Nikoleta-v3 commented Jul 20, 2024 • edited Loading

Paper

Tests

Nikoleta-v3 commented Jul 20, 2024

rwestaway commented Jul 22, 2024

Paper

Tests

Paper

Tests

mnky9800n commented Jul 22, 2024

PieterjanRobbe commented Jul 23, 2024

Nikoleta-v3 commented Jul 23, 2024

Paper

Tests

Nikoleta-v3 commented Jul 23, 2024

Nikoleta-v3 commented Jul 23, 2024

editorialbot commented Jul 23, 2024

rwestaway commented Jul 24, 2024

editorialbot commented Jul 24, 2024

editorialbot commented Jul 24, 2024

wcjochem commented Jul 25, 2024

rwestaway commented Jul 26, 2024

Tests

editorialbot commented Jul 26, 2024

editorialbot commented Jul 26, 2024

editorialbot commented Jul 26, 2024

editorialbot commented Jul 26, 2024

Nikoleta-v3 commented Jul 28, 2024

Nikoleta-v3 commented Jul 28, 2024

editorialbot commented Jul 28, 2024

Nikoleta-v3 commented Jul 28, 2024

editorialbot commented Jul 28, 2024

editorialbot commented Jul 16, 2024 •

edited

Loading

rwestaway commented Jul 16, 2024 •

edited

Loading

Nikoleta-v3 commented Jul 20, 2024 •

edited

Loading