[PRE REVIEW]: MPI-Rockstar: a Hybrid MPI and OpenMP Parallel Implementation of the Rockstar Halo finder

[editorialbot]

Submitting author: @Tomoaki-Ishiyama (Tomoaki Ishiyama)
Repository: https://github.com/Tomoaki-Ishiyama/mpi-rockstar
Branch with paper.md (empty if default branch): joss
Version: v1.0.0
Editor: Pending
Reviewers: Pending
Managing EiC: Warrick Ball

Status

Status badge code:

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

Author instructions

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

@Tomoaki-Ishiyama 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

[editorialbot]

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]

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

✅ OK DOIs

- 10.1093/pasj/61.6.1319 is OK
- 10.1093/mnras/stab1755 is OK
- 10.1186/s40668-017-0021-1 is OK
- 10.1093/mnras/stac3072 is OK
- 10.1088/0004-637X/762/2/109 is OK
- 10.1088/1674-4527/18/6/62 is OK
- 10.1093/mnras/stt1403 is OK
- 10.1017/pasa.2019.12 is OK
- 10.1093/mnras/stab2482 is OK
- 10.1093/mnras/stab1855 is OK
- 10.1088/0067-0049/182/2/608 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: 4.45 Pflops astrophysical N-body simulation on K c...

❌ MISSING DOIs

- None

❌ INVALID DOIs

- None

[editorialbot]

Software report:

github.com/AlDanial/cloc v 1.98  T=0.05 s (1830.8 files/s, 318894.7 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
C                               42           1253            377          10168
C++                              1            321             76           1633
C/C++ Header                    44            286             68           1357
TeX                              1             32              0            225
Markdown                         2             47              0            197
Perl                             1             27              5            178
YAML                             2              4              8             46
make                             1             16              3             46
-------------------------------------------------------------------------------
SUM:                            94           1986            537          13850
-------------------------------------------------------------------------------

Commit count by author:

    58	Tomoaki-Ishiyama
    24	Ken Osato
     2	stanaka

[editorialbot]

Paper file info:

📄 Wordcount for paper.md is 584

✅ The paper includes a Statement of need section

[editorialbot]

License info:

🔴 Failed to discover a valid open source license

[editorialbot]

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

[editorialbot]

Five most similar historical JOSS papers:

CosmicProfiles: A Python package for radial profiling of finitely sampled dark matter halos and galaxies
Submitting author: @tibordome
Handling editor: @adonath (Active)
Reviewers: @benediktdiemer, @phil-mansfield
Similarity score: 0.7395

Nyx: A Massively Parallel AMR Code for Computational Cosmology
Submitting author: @jmsexton03
Handling editor: @dfm (Active)
Reviewers: @c-white, @teuben
Similarity score: 0.7261

MARTINI: Mock Array Radio Telescope Interferometry of the Neutral ISM
Submitting author: @kyleaoman
Handling editor: @dfm (Active)
Reviewers: @dfm
Similarity score: 0.7244

MAESTROeX: A Massively Parallel Low Mach Number Astrophysical Solver
Submitting author: @doreenfan
Handling editor: @arfon (Active)
Reviewers: @tukss, @pgrete
Similarity score: 0.7195

Paicos: A Python package for analysis of (cosmological) simulations performed with Arepo
Submitting author: @tberlok
Handling editor: @JBorrow (Active)
Reviewers: @ttricco, @kyleaoman
Similarity score: 0.7182

⚠️ 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.

[warrickball]

Hi @Tomoaki-Ishiyama, and thanks for your submission to JOSS. The nature of this submission looks a bit unusual so I'd like to clear up a few things before we consider continuing to review. Specifically, is the idea that JOSS reviews the submitted repo and the original Rockstar code? Or is this submission in effect a standalone fork that extends the previous code?

The latter is in principle in scope for JOSS (see Should I write my own software or contribute to an existing package?) but I notice that the new documentation is just the top-level README, which has some technical details but little context, and there don't appear to be any examples ready to run out-of-the-box. You might find inspiration for the kind of documentation and examples we strive for by glancing at some of the similar articles listed in the comment above.