[PRE REVIEW]: PyLogGrid: A Python package for fluid dynamics on logarithmic lattices

[editorialbot]

Submitting author: @hippalectryon-0 (Amaury Barral)
Repository: https://github.com/hippalectryon-0/pyloggrid
Branch with paper.md (empty if default branch): joss
Version: 2.3.0
Editor: @philipcardiff
Reviewers: @slaizet, @marlonsmathias, @weiwangstfc
Managing EiC: Arfon Smith

Status

Status badge code:

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

Author instructions

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

@hippalectryon-0 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]

Software report:

github.com/AlDanial/cloc v 1.88  T=0.39 s (210.1 files/s, 23656.0 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          30           1136           1093           3533
Markdown                         9            152              0            388
reStructuredText                16            340            207            347
Jupyter Notebook                 1              0            344            320
YAML                             5             24             30            242
Cython                           1             22              9            218
C                                1             26              0            169
TeX                              1             13              0             96
PowerShell                       2             17             13             81
Bourne Shell                     5             17             15             70
TOML                             1              9              1             69
SVG                              5              1              1             31
make                             2              9              9             29
DOS Batch                        2              8              1             28
-------------------------------------------------------------------------------
SUM:                            81           1774           1723           5621
-------------------------------------------------------------------------------


gitinspector failed to run statistical information for the repository

[editorialbot]

Wordcount for paper.md is 499

[editorialbot]

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

OK DOIs

- 10.1146/annurev.fluid.35.101101.161122 is OK
- 10.1103/PhysRevLett.77.5369 is OK
- 10.48550/arXiv.2006.00047 is OK
- 10.1017/jfm.2023.204 is OK
- 10.3390/atmos14111690 is OK

MISSING DOIs

- 10.1103/physreve.107.065106 may be a valid DOI for title: Reversible Navier-Stokes equation on logarithmic lattices
- 10.1016/j.jcp.2014.09.038 may be a valid DOI for title: Exponential time-differencing with embedded Runge–Kutta adaptive step control

INVALID DOIs

- https://doi.org/10.1016/0167-2789(85)90002-8 is INVALID because of 'https://doi.org/' prefix

[editorialbot]

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

[editorialbot]

Five most similar historical JOSS papers:

Capytaine: a Python-based linear potential flow solver
Submitting author: @mancellin
Handling editor: @leouieda (Retired)
Reviewers: @mikaem, @harpolea
Similarity score: 0.8138

VlaPy: A Python package for Eulerian Vlasov-Poisson-Fokker-Planck Simulations
Submitting author: @joglekara
Handling editor: @dpsanders (Retired)
Reviewers: @TomGoffrey, @StanczakDominik
Similarity score: 0.8126

lattice_mc: A Python Lattice-Gas Monte Carlo Module
Submitting author: @bjmorgan
Handling editor: @labarba (Retired)
Reviewers: @asmit3
Similarity score: 0.8046

PyAMG: Algebraic Multigrid Solvers in Python
Submitting author: @lukeolson
Handling editor: @jedbrown (Active)
Reviewers: @mayrmt, @mattmartineau
Similarity score: 0.8038

pyro: a framework for hydrodynamics explorations and prototyping
Submitting author: @harpolea
Handling editor: @labarba (Retired)
Reviewers: @mikaem, @ngoldbaum
Similarity score: 0.8034

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

[kyleniemeyer]

Hi @hippalectryon-0, sorry for the delay in updates here. We are waiting for an editor to become available to handle your submission, which should hopefully happen soon.

[philipcardiff]

@editorialbot assign me as editor

[editorialbot]

Assigned! @philipcardiff is now the editor

[philipcardiff]

Hello @hippalectryon-0, do you have any suggestions for appropriate reviewers? Maybe even from the list of similar historical JOSS papers?

[philipcardiff]

FYI, I invited Ciro S. Campolina to review this submission via email (and am awaiting a reply), since you refer to their LogLatt library.

[hippalectryon-0]

@philipcardiff
Other than C. Campolina, I don't really have relevant reviewers from my field to suggest, as most aren't really in the numerical side of things.
Looking quickly are other JOSS fluid dynamics paper, no reviewers really ring a bell, so I'm not sure that I can make a very relevant suggestion. Just in case, here are a few reviewers that reviewed somewhat similar (?) fluid dynamics papers (I removed the "@" to not ping them all): ashwinvis, NoraLoose, ranocha, eviatarbach, DamienIrving

[philipcardiff]

Hi @hippalectryon-0,

Would you mind addressing the following points on the documentation (https://pyloggrid.readthedocs.io/en/latest/benchmark.html#vs-grid-size):

• Fix missing images at https://pyloggrid.readthedocs.io/en/latest/benchmark.html#vs-grid-size
• Fix missing images at https://pyloggrid.readthedocs.io/en/latest/tutorial/tutorial_analyzing_output.html
• Fix typo at https://pyloggrid.readthedocs.io/en/latest/documentation/doc_solvers.html: veru stiff => very stiff
• Fix missing images at https://pyloggrid.readthedocs.io/en/latest/documentation/doc_performance.html
• Fix missing images at https://pyloggrid.readthedocs.io/en/latest/documentation/doc_plotting.html

Also, when I go to the main repository (https://github.com/hippalectryon-0/pyloggrid), the README file says this is not the main repository and it may not always be up to date; this is strange since the review is presumably meant to be on the main repository. At a minimum, please provide a link to the documentation in the README.

I am still searching for suitable reviewers.

[philipcardiff]

For the tutorial at https://pyloggrid.readthedocs.io/en/latest/tutorial/tutorial_first_simu.html#running-the-simulation, can you add more details on the problem definition? e.g.

• Geometry
• Material properties
• Loading/boundary conditions
• Initial conditions

This would also help me find appropriate reviewers as I would better understand what type of problems PyLogGrid can solve.

Thanks.

[philipcardiff]

HI @slaizet,

Would you be interested in reviewing the submission “PyLogGrid: A Python package for fluid dynamics on logarithmic lattice”, which we received at JOSS? The link for the submission is #6087 , and the repository is at https://github.com/hippalectryon-0/pyloggrid.

If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected.

[philipcardiff]

Hi @friedenhe,

Would you be interested in reviewing the submission “PyLogGrid: A Python package for fluid dynamics on logarithmic lattice”, which we received at JOSS? The link for the submission is #6087 , and the repository is at https://github.com/hippalectryon-0/pyloggrid.

If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected

[slaizet]

Happy to review this submission!

[philipcardiff]

Happy to review this submission!

Great, thank you.

[philipcardiff]

@editorialbot add @slaizet as reviewer

[editorialbot]

@slaizet added to the reviewers list!

[friedenhe]

@philipcardiff Thanks very much for the invitation. However, I am at my maximum bandwidth now, and I am afraid that I won't be able to review this paper.

[philipcardiff]

@philipcardiff Thanks very much for the invitation. However, I am at my maximum bandwidth now, and I am afraid that I won't be able to review this paper.

No problem, thanks for letting me know.

[philipcardiff]

Hi @p-costa,

Would you be interested in reviewing the submission “PyLogGrid: A Python package for fluid dynamics on logarithmic lattice”, which we received at JOSS? The link for the submission is #6087 , and the repository is at https://github.com/hippalectryon-0/pyloggrid.

If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected

[p-costa]

Thanks @philipcardiff for the invite! Unfortunately I don't have time to review this nice submission, as I am on parental leave.

[philipcardiff]

Thanks @philipcardiff for the invite! Unfortunately I don't have time to review this nice submission, as I am on parental leave.

No problem; thanks for your quick reply!

[philipcardiff]

HI @GongZheng-Justin, Would you be interested in reviewing the submission “PyLogGrid: A Python package for fluid dynamics on logarithmic lattice”, which we received at JOSS? The link for the submission is #6087 , and the repository is at https://github.com/hippalectryon-0/pyloggrid.

If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected.

[philipcardiff]

HI @davecats, Would you be interested in reviewing the submission “PyLogGrid: A Python package for fluid dynamics on logarithmic lattice”, which we received at JOSS? The link for the submission is #6087 , and the repository is at https://github.com/hippalectryon-0/pyloggrid.

If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected.

[slaizet]

Hi @hippalectryon-0,

I tried to follow your instructions to clone the code but I got an error message:

git clone [email protected]:hippalectryon-0/pyloggrid.git
Cloning into 'pyloggrid'...
The authenticity of host 'github.com (2a0c:5bc0:40:2fff::8c52:7903)' can't be established.
ECDSA key fingerprint is XXX.
Are you sure you want to continue connecting (yes/no/[fingerprint])? yes
Warning: Permanently added 'github.com,2a0c:5bc0:40:2fff::8c52:7903' (ECDSA) to the list of known hosts.
[email protected]: Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Can you double check that the instructions are correct?

Cheers,

Sylvain

[hippalectryon-0]

@slaizet Can you try git clone https://github.com/hippalectryon-0/pyloggrid.git instead ?

(note that you'll then need to run cd pyloggrid && git checkout joss to switch to the joss branch)

[slaizet]

All good now with the new instructions (you might want to update your doc).
Thanks!

[hippalectryon-0]

For the tutorial at https://pyloggrid.readthedocs.io/en/latest/tutorial/tutorial_first_simu.html#running-the-simulation, can you add more details on the problem definition? e.g.

* Geometry

* Material properties

* Loading/boundary conditions

* Initial conditions

This would also help me find appropriate reviewers as I would better understand what type of problems PyLogGrid can solve.

Thanks.

As requested I've added some more information in the tutorial. I've also updated the docs to reflect the issue @slaizet faced.

(if you still see the old docs, 1) check that you're on the joss branch of the docs 2) clear your browser cache)

[slaizet]

Hi @hippalectryon-0,

I am trying to run my "first simulation" following the instructions here: https://pyloggrid.readthedocs.io/en/latest/tutorial/tutorial_first_simu.html

First of all, you should specify all the Python libraries required to run NS3D.py. Secondly, I get an error message when trying to run the code with python:

File "NS3D.py", line 22
    def get_forcing(grid: Grid) -> tuple[np.ndarray, np.ndarray, np.ndarray]:
                        ^
SyntaxError: invalid syntax

I get a different error when trying to run the code with python3:

Traceback (most recent call last):
  File "NS3D.py", line 14, in <module>
    from pyloggrid.Libs.datasci import randcomplex_like, randcomplex_seeded_by_array
  File "/home/slaizet/reviewJOSS/pyloggrid/log-grid/Simulations/../pyloggrid/Libs/datasci.py", line 33, in <module>
    def mean(data: np.ndarray, ts: Optional[np.ndarray] = None, log: bool = False, start: float = 0, end: float = 1) -> tuple[np.ndarray, np.ndarray | None]:
TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

I can confirm that I am in the joss branch:

On branch joss
Your branch is up-to-date with 'origin/joss'.

Am I missing something?

Cheers,

Sylvain

[philipcardiff]

Hello @weiwangstfc, Would you be interested in reviewing the submission “PyLogGrid: A Python package for fluid dynamics on logarithmic lattice”, which we received at JOSS? The link for the submission is #6087 , and the repository is at https://github.com/hippalectryon-0/pyloggrid.

If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected.

[philipcardiff]

Hi @marlonsmathias, Would you be interested in reviewing the submission “PyLogGrid: A Python package for fluid dynamics on logarithmic lattice”, which we received at JOSS? The link for the submission is #6087 , and the repository is at https://github.com/hippalectryon-0/pyloggrid.

If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected.

[marlonsmathias]

Hello @philipcardiff , yes, I am interested in reviewing the submission. I have not reviewed for JOSS yet, so I will take a look at the documentation.

[philipcardiff]

Hello @philipcardiff , yes, I am interested in reviewing the submission. I have not reviewed for JOSS yet, so I will take a look at the documentation.

Thanks, @marlonsmathias.

[philipcardiff]

@editorialbot add @marlonsmathias as reviewer

[editorialbot]

@marlonsmathias added to the reviewers list!

[philipcardiff]

@editorialbot start review

[editorialbot]

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

[philipcardiff]

Hi all, we can continue the review over on the review issue #6439.

[weiwangstfc]

Hi Philip, I can review the mentioned submission if you are still looking for reviewers. Thanks. Regards, Wei

…

________________________________ From: Philip Cardiff ***@***.***> Sent: Friday, March 1, 2024 15:27 To: openjournals/joss-reviews ***@***.***> Cc: Wang, Wei (STFC,DL,SC) ***@***.***>; Mention ***@***.***> Subject: Re: [openjournals/joss-reviews] [PRE REVIEW]: PyLogGrid: A Python package for fluid dynamics on logarithmic lattices (Issue #6087) Hello @weiwangstfc<https://github.com/weiwangstfc>, Would you be interested in reviewing the submission “PyLogGrid: A Python package for fluid dynamics on logarithmic lattice”, which we received at JOSS? The link for the submission is #6087<#6087> , and the repository is at https://github.com/hippalectryon-0/pyloggrid. If you are unfamiliar with JOSS, the review process is unique: it takes place in a GitHub issue, is open, and encourages author-reviewer-editor conversations. You can have a look at our review criteria (https://joss.readthedocs.io/en/latest/review_criteria.html) and review checklist (https://joss.readthedocs.io/en/latest/review_checklist.html) to get an idea of what is expected. — Reply to this email directly, view it on GitHub<#6087 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ARMNZ247CJNMQ6CV32UNUKDYWCM7VAVCNFSM6AAAAAA7Y2LER2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZTGM4DOMRTGM>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[philipcardiff]

Hi Philip, I can review the mentioned submission if you are still looking for reviewers. Thanks. Regards, Wei
…

Great, thanks Wei.

[philipcardiff]

@editorialbot add @weiwangstfc as reviewer

[editorialbot]

@weiwangstfc added to the reviewers list!

[hippalectryon-0]

@slaizet
Apologies for the late answer, I missed the notification.

First of all, you should specify all the Python libraries required to run NS3D.py. Secondly, I get an error message when trying to run the code with python:

Could you confirm that you have successfully ran setup.sh as indicated here ? Your error indicates that you're using a python version which is too old. The setup takes care of installing the right python version, the virtual environment, and all the required packages.