openjournals / joss-reviews

Reviews for the Journal of Open Source Software
Creative Commons Zero v1.0 Universal
707 stars 37 forks source link

[PRE REVIEW]: CASTRO: A Massively Parallel Compressible Astrophysics Simulation Code #2468

Closed whedon closed 4 years ago

whedon commented 4 years ago

Submitting author: @zingale (M. Zingale) Repository: https://github.com/amrex-astro/Castro Version: 20.07 Editor: @eloisabentivegna Reviewers: @kegiljarhus, @bonh Managing EiC: Kyle Niemeyer

:warning: JOSS reduced service mode :warning:

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

Author instructions

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

@zingale if you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). In addition, this list of people have already agreed to review for JOSS and may be suitable for this submission (please start at the bottom of the list).

Editor instructions

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

@whedon commands
whedon commented 4 years ago

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

:warning: JOSS reduced service mode :warning:

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

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

@whedon commands

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

@whedon generate pdf
whedon commented 4 years ago

Failed to discover a Statement of need section in paper

whedon commented 4 years ago

PDF failed to compile for issue #2468 with the following error:

pandoc-citeproc: reference ctu not found pandoc-citeproc: reference ppm not found Error producing PDF. ! Undefined control sequence. l.431 ... research. The GPU development of \castro

Looks like we failed to compile the PDF

whedon commented 4 years ago
Software report (experimental):

github.com/AlDanial/cloc v 1.84  T=1.87 s (460.8 files/s, 86976.5 lines/s)
--------------------------------------------------------------------------------
Language                      files          blank        comment           code
--------------------------------------------------------------------------------
C++                             111          11484           5015          36472
Fortran 90                      222          12858           5209          35233
Python                          110           3998           1481           9471
C/C++ Header                    103           2437           3929           7229
reStructuredText                 36           2904           1083           6994
Markdown                         33            675              0           2178
make                             81            889            198           1370
TeX                               4            134              6           1273
Bourne Shell                     64            561            241           1246
SVG                               5              5              5            850
Fortran 77                        1            353            298            719
Bourne Again Shell               71            447            752            652
INI                               2             89              0            628
Modula3                           2             83              0            300
Jupyter Notebook                  5              0           1975            161
Korn Shell                        2             76             81            107
YAML                              3             22              7            105
ANTLR Grammar                     1             19              0             62
JSON                              1              0              0             52
HTML                              3              4              5             46
CSS                               1              9              5             36
--------------------------------------------------------------------------------
SUM:                            861          37047          20290         105184
--------------------------------------------------------------------------------

Statistical information for the repository '2468' was gathered on 2020/07/10.
The following historical commit information, by author, was found:

Author                     Commits    Insertions      Deletions    % of changes
Abigail Bishop                   1            34              2            0.01
Alice Harpole                   43          3707            214            0.79
Andrew Myers                     6            76             59            0.03
Ann Almgren                    145         21258          16014            7.53
Brian Friesen                    1             1              1            0.00
Chris DeGrendele                 2           325              1            0.07
Chris Malone                    20           186             82            0.05
Christopher Degrende             1           329              4            0.07
Cy P Chan                        1             1              2            0.00
Donald E. Willcox                8           135             26            0.03
KiranEiden                       6          1268            304            0.32
Marc Day                         2            18              0            0.00
Maria Barrios Sazo              12          1050            124            0.24
Max Katz                      1578         48881          38275           17.60
Michael Zingale                891        165464         122355           58.13
Mike Lijewski                    2             0              4            0.00
WeiqunZhang                    537         52941          18585           14.45
blaireness                       2          1002             23            0.21
jbb                              2             4              2            0.00
jmsexton03                       2           724            185            0.18
taehoryu                         7           992            207            0.24
vince                            2           242              2            0.05

Below are the number of rows from each author that have survived and are still
intact in the current revision:

Author                     Rows      Stability          Age       % in comments
Abigail Bishop               30           88.2         21.3                6.67
Alice Harpole              3488           94.1         14.6               13.73
Andrew Myers                 51           67.1         40.2                7.84
Ann Almgren                1506            7.1         92.5                9.63
Chris DeGrendele            325          100.0         17.0                5.85
Chris Malone                  3            1.6         95.5                0.00
Christopher Degrende          5            1.5         15.7                0.00
Cy P Chan                     1          100.0         18.5                0.00
Donald E. Willcox           116           85.9         21.3                6.03
KiranEiden                  964           76.0         16.6                9.23
Marc Day                      9           50.0         93.7               11.11
Maria Barrios Sazo          870           82.9          3.1                2.87
Max Katz                  15428           31.6         21.6                9.18
Michael Zingale           29804           18.0         20.4                9.94
WeiqunZhang               13581           25.7         56.8                8.00
blaireness                 1002          100.0         22.5               16.67
jmsexton03                  702           97.0          2.8                7.12
taehoryu                    992          100.0         34.3                3.12
vince                        71           29.3         82.2                0.00
kyleniemeyer commented 4 years ago

Hi @zingale, thanks for your submission!

Before we get started, I just want to check that you haven't already published a software paper describing CASTRO. (I recall seeing some papers about CASTRO in the past.) Papers using CASTRO are fine, but if you have already published a software article about CASTRO, then we would consider this to be a duplication. Exceptions would be if this represents a substantially new/different version.

zingale commented 4 years ago

I guess this depends on the difference between a code paper and an algorithm paper. We've mostly published on new algorithm development and performance and optimization. The only paper that covers all of Castro in general is:

There was a paper II and III in that series that focused on radiation and neutrino transport. The code has changed a lot since 2010, and many new developers have joined that do not receive credit from citing that 2010 paper.

We've had a number of conference proceedings and focused algorithm papers:

In addition to those, there are a number of science papers that contain appendices that include some verification of a particular part of the algorithm.

The reason we are submitting to JOSS is there is no up-to-date paper to cite for the present version of the code from the current developer groups. In general, we only cite the 2010 ApJ paper in our science paper.

We've never had a paper where the reviewers looked at the source code, documentation, and verified that it ran as we describe, which is our motivation for submitting to JOSS.

kyleniemeyer commented 4 years ago

@zingale ok, thanks. That clears the question up for me, at least.

Hi @eloisabentivegna, can you edit this submission?

kyleniemeyer commented 4 years ago

@whedon invite @eloisabentivegna as editor

whedon commented 4 years ago

@eloisabentivegna has been invited to edit this submission.

zingale commented 4 years ago

it's a fine question to ask @kyleniemeyer .

zingale commented 4 years ago

some potential reviewers:

jramirezcr mikaem bonh kegiljarhus joshia5 fhorrobin mdelorme

arfon commented 4 years ago

@zingale - does this paper accompany a submission to a AAS journal, i.e. is this part of https://blog.joss.theoj.org/2018/12/a-new-collaboration-with-aas-publishing

zingale commented 4 years ago

no. The closest AAS paper to this is from 2010.

eloisabentivegna commented 4 years ago

@whedon assign @eloisabentivegna as editor

whedon commented 4 years ago

OK, the editor is @eloisabentivegna

eloisabentivegna commented 4 years ago

@whedon add @kegiljarhus as reviewer

whedon commented 4 years ago

OK, @kegiljarhus is now a reviewer

eloisabentivegna commented 4 years ago

@whedon add @bonh as reviewer

whedon commented 4 years ago

OK, @bonh is now a reviewer

eloisabentivegna commented 4 years ago

@whedon start review

whedon commented 4 years ago

OK, I've started the review over in https://github.com/openjournals/joss-reviews/issues/2513.