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

[PRE REVIEW]: GECo: A collection of solvers for the self-gravitating Vlasov equations #5859

Closed
editorialbot opened this issue Sep 20, 2023 · 25 comments
Assignees
Labels
Dockerfile pre-review Python Shell Track: 1 (AASS) Astronomy, Astrophysics, and Space Sciences

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Sep 20, 2023

Submitting author: @alogg (Anders Logg)
Repository: https://github.com/anderslogg/geco
Branch with paper.md (empty if default branch): paper-joss-2023
Version: 1.0.0
Editor: @dfm
Reviewers: @michellegurevich, @pmocz
Managing EiC: Dan Foreman-Mackey

Status

status

Status badge code:

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

Author instructions

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

@alogg 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 editorialbot added pre-review Track: 1 (AASS) Astronomy, Astrophysics, and Space Sciences labels Sep 20, 2023
@editorialbot
Copy link
Collaborator Author

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
Copy link
Collaborator Author

Software report:

github.com/AlDanial/cloc v 1.88  T=0.06 s (1462.5 files/s, 133442.2 lines/s)
--------------------------------------------------------------------------------
Language                      files          blank        comment           code
--------------------------------------------------------------------------------
Python                           47           1332           1268           3350
C/C++ Header                     22            326            295            994
Markdown                          5             57              0            216
Jupyter Notebook                  2              0            306             84
Bourne Shell                      7             25             35             78
TeX                               1              6              0             59
Bourne Again Shell                5             14             14             39
reStructuredText                  1             11              0             25
Dockerfile                        1              8              9             20
GLSL                              1              0              1              2
make                              1              0              0              2
XML                               1              0              0              1
--------------------------------------------------------------------------------
SUM:                             94           1779           1928           4870
--------------------------------------------------------------------------------


gitinspector failed to run statistical information for the repository

@editorialbot
Copy link
Collaborator Author

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

OK DOIs

- 10.1007/978-3-642-23099-8 is OK
- 10.1088/0264-9381/33/15/155008 is OK
- 10.12942/lrr-2011-4 is OK
- 10.1103/PhysRevD.77.124044 is OK
- 10.1137/10078356X is OK

MISSING DOIs

- None

INVALID DOIs

- None

@editorialbot
Copy link
Collaborator Author

Wordcount for paper.md is 908

@editorialbot
Copy link
Collaborator Author

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

@dfm
Copy link

dfm commented Sep 21, 2023

@alogg — Thanks for your submission! All the suitable JOSS editors are currently working at capacity so I'm going to "waitlist" this review until an editor with the relevant expertise is available to take it on. Thanks for your patience!

@dfm dfm added the waitlisted Submissions in the JOSS backlog due to reduced service mode. label Sep 21, 2023
@elleryames
Copy link

Thanks for the note @dfm and thanks to JOSS for considering our contribution. A few suitable reviewers are (github handles without @ ): KAClough, sbozzolo, shreyasbapat, and apoudel2014.

@dfm
Copy link

dfm commented Oct 16, 2023

@editorialbot assign me as editor

@elleryames, @alogg — Thanks for your patience. I now have capacity to edit your submission so I'll take it from here. I'll start reaching out to reviewers now, and then once we have 2 assigned, we'll get the main review started in a new thread.

@editorialbot
Copy link
Collaborator Author

Assigned! @dfm is now the editor

@dfm dfm removed the waitlisted Submissions in the JOSS backlog due to reduced service mode. label Oct 16, 2023
@dfm
Copy link

dfm commented Oct 16, 2023

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

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

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

GRChombo: An adaptable numerical relativity code for fundamental physics
Submitting author: @kaclough
Handling editor: @rkurchin (Active)
Reviewers: @sbozzolo, @rashti-alireza
Similarity score: 0.8277

GeophysicalFlows.jl: Solvers for geophysical fluid dynamics problems in periodic domains on CPUs & GPUs
Submitting author: @navidcy
Handling editor: @pdebuyl (Active)
Reviewers: @ranocha, @eviatarbach
Similarity score: 0.8273

UWGeodynamics: A teaching and research tool for numerical geodynamic modelling
Submitting author: @rbeucher
Handling editor: @lheagy (Retired)
Reviewers: @flohorovicic
Similarity score: 0.8150

ESVM: an open-source finite volume Electrostatic Vlasov-Maxwell code
Submitting author: @michaeltouati
Handling editor: @danielskatz (Active)
Reviewers: @tclune, @TomGoffrey, @rouson
Similarity score: 0.8132

CASTRO: A Massively Parallel Compressible Astrophysics Simulation Code
Submitting author: @zingale
Handling editor: @eloisabentivegna (Active)
Reviewers: @kegiljarhus, @bonh, @joshia5
Similarity score: 0.8111

⚠️ 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 before considering asking the reviewers of these papers to review again for JOSS.

@dfm
Copy link

dfm commented Oct 16, 2023

@elleryames, @alogg — In the meantime it looks like most of the references in the compiled document are not getting resolved. Can you check your BibTeX and try to work out what's going on? There are some methods outlined on the JOSS docs page to check the compilation. Thanks!

@anderslogg
Copy link

@dfm We have followed the instructions on the web page very carefully but still the figure references are broken.

The instructions on the web page look like this:

![Caption for example figure.\label{fig:example}](figure.png)
and referenced from text using \autoref{fig:example}.

And we have the following in our paper:

\autoref{fig:Solution}

![Torus spatial density on quarter plane computational domain \label{fig:Solution}](./figures/density_computational_domain.png)

We have also built the document using the Docker instructions as listed in the instruction but the references break for some reason.

@dfm
Copy link

dfm commented Oct 16, 2023

@anderslogg — I've opened a PR which fixes the references, but I think you'll also want to write somewhat more descriptive captions :D

@anderslogg
Copy link

@dfm Thanks! Will fix. So it was just a matter of separating the figures with newlines?

@dfm
Copy link

dfm commented Oct 16, 2023

Great! Yes, that seems to have been sufficient.

@anderslogg
Copy link

@dfm Merged pull request and fixed captions now.

@dfm
Copy link

dfm commented Oct 19, 2023

@editorialbot add @michellegurevich as reviewer

🎉 Thanks to @michellegurevich for agreeing to act as a reviewer for this submission! As soon as we have a second reviewer, I'll get the main review thread started.

@editorialbot
Copy link
Collaborator Author

@michellegurevich added to the reviewers list!

@dfm
Copy link

dfm commented Oct 23, 2023

@editorialbot add @pmocz as reviewer

🎉 Thanks @pmocz for agreeing to be our second reviewer!

Before I start the review thread, I want to confirm: @anderslogg is @alogg also your GitHub username? If so, which do you prefer, and could you update your JOSS profile page if you'd like @anderslogg to be the one we use?

@editorialbot
Copy link
Collaborator Author

@pmocz added to the reviewers list!

@anderslogg
Copy link

@dfm Yes that's an old account. Profile updated with @anderslogg (new main account) now.

@dfm
Copy link

dfm commented Oct 23, 2023

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dockerfile pre-review Python Shell Track: 1 (AASS) Astronomy, Astrophysics, and Space Sciences
Projects
None yet
Development

No branches or pull requests

4 participants