Skip to content
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

[DO NOT MERGE] Paper review comments #84

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gassmoeller
Copy link

These are my review comments for the JOSS paper draft. This PR is not meant to be merged, it was just the easiest way for me to comment on specific pieces of the paper. I enjoyed reading the paper it is well written and covers the most important topics of a JOSS paper. However, I think the paper should be extended somewhat to include:

  • more references to existing benchmark studies to provide a better view of the state of the field
  • more references to other pre/postprocessing tools to establish where your solution fits into existing projects
  • a more detailed description of the algorithms you use (in particular for the postprocessing interpolation or mapping)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant