sbocce / hyper2D

Finite volume solver for hyperbolic PDEs and hypersonic flows
BSD 3-Clause "New" or "Revised" License
19 stars 2 forks source link

Add contribution guidelines #3

Closed ketch closed 2 years ago

ketch commented 2 years ago

To save time, it would be very helpful if you read through what is expected for publication in JOSE and complete all the missing items, and then ping me to continue with the review. You can see what is needed for example here.

sbocce commented 2 years ago

Hello, thanks a lot for your inputs & for accepting to review the submission. I'll follow your suggestions and will let you know as soon as I'm done.

sbocce commented 2 years ago

Hello @ketch, I have modified the paper & repository following the instructions. Here's a list of modifications and replies to the instructions.

Functionality:

Documentation:

Software paper

Let me know if you need some further action on my side.

ketch commented 2 years ago

It seems that the main documentation is the Github wiki, but I don't see how anyone would possibly know to look there, except by reading this issue thread. Perhaps you should state in some prominent places that the wiki is the documentation, with a link.

ketch commented 2 years ago

I find it very strange that the package targets beginners, but doesn't give any suggestions on how to plot the output. Your introductory text mentions "good looking fluid dynamics simulations", but your package does not produce anything "good looking", right?

Additionally, saying that the output is "a simple dat file" is not helpful, since this doesn't mean anything (at least, to me). Is it ASCII? Is it binary? What is the format? How should one read it?

ketch commented 2 years ago

In the installation instructions, asking the user as a first step to browse the directory structure and find one with a Makefile is okay, but not great in my opinion. It would be much better if you listed the directory for each package in the README.