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]: libtrixi: an interface library for using Trixi.jl from C/C++/Fortran #5943

Closed
editorialbot opened this issue Oct 13, 2023 · 18 comments
Labels
CMake Julia Makefile pre-review query-scope Submissions of uncertain scope for JOSS rejected Track: 7 (CSISM) Computer science, Information Science, and Mathematics

Comments

@editorialbot
Copy link
Collaborator

Submitting author: @sloede (Michael Schlottke-Lakemper)
Repository: https://github.com/trixi-framework/libtrixi
Branch with paper.md (empty if default branch): paper-2023-joss
Version: v0.1.4
Editor: Pending
Reviewers: Pending
Managing EiC: Daniel S. Katz

Status

status

Status badge code:

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

Author instructions

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

@sloede 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: 7 (CSISM) Computer science, Information Science, and Mathematics labels Oct 13, 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.04 s (1883.8 files/s, 182022.4 lines/s)
--------------------------------------------------------------------------------
Language                      files          blank        comment           code
--------------------------------------------------------------------------------
Markdown                         10            194              0            940
Julia                            16            375            190            855
C                                 7            211            376            427
CMake                             7            139            100            409
Fortran 90                        7            135            353            382
Bourne Again Shell                1             37             25            318
C++                               4             87             72            229
XML                               1              6              8            226
TeX                               2             11              0            151
TOML                              5              9              5             52
YAML                              3              2              6             39
C/C++ Header                      2             16             18             37
JSON                              1              0              0             24
make                              1              3              0              7
--------------------------------------------------------------------------------
SUM:                             67           1225           1153           4096
--------------------------------------------------------------------------------


gitinspector failed to run statistical information for the repository

@editorialbot
Copy link
Collaborator Author

Wordcount for paper.md is 1317

@editorialbot
Copy link
Collaborator Author

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

OK DOIs

- 10.21105/jcon.00077 is OK
- 10.1016/j.jcp.2021.110467 is OK
- 10.5281/zenodo.3996439 is OK
- 10.5281/zenodo.8321803 is OK
- 10.1137/141000671 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.1007/978-1-4612-1986-6_8 is OK
- 10.1016/j.camwa.2020.02.022 is OK
- 10.21105/joss.02520 is OK
- 10.21105/joss.04644 is OK

MISSING DOIs

- None

INVALID DOIs

- None

@editorialbot
Copy link
Collaborator Author

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

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

gtk-fortran: a GTK+ binding to build Graphical User Interfaces in Fortran
Submitting author: @vmagnin
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @johnsamuelwrites, @sundmanbo
Similarity score: 0.8025

Flux: Elegant machine learning with Julia
Submitting author: @MikeInnes
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @ysimillides
Similarity score: 0.8021

Optim: A mathematical optimization package for Julia
Submitting author: @anriseth
Handling editor: @labarba (Retired)
Reviewers: @ahwillia
Similarity score: 0.8002

Gridap: An extensible Finite Element toolbox in Julia
Submitting author: @fverdugo
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @PetrKryslUCSD, @TeroFrondelius, @KristofferC
Similarity score: 0.7958

Kinetic.jl: A portable finite volume toolbox for scientific and neural computing
Submitting author: @vavrines
Handling editor: @diehlpk (Active)
Reviewers: @rdeits, @jarvist
Similarity score: 0.7946

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

@danielskatz
Copy link

👋 @sloede - thanks for your submission. I'm going to ask the JOSS editors to examine its suitability for review in JOSS, since as an interface library, it's somewhat a wrapper, which might be out of scope for us. You should hear back in a week or two.

@danielskatz
Copy link

@editorialbot query scope

@editorialbot
Copy link
Collaborator Author

Submission flagged for editorial review.

@editorialbot editorialbot added the query-scope Submissions of uncertain scope for JOSS label Oct 13, 2023
@sloede
Copy link

sloede commented Oct 13, 2023

👋 @sloede - thanks for your submission. I'm going to ask the JOSS editors to examine its suitability for review in JOSS, since as an interface library, it's somewhat a wrapper, which might be out of scope for us. You should hear back in a week or two.

Thanks a lot for the heads up, @danielskatz! Please note that we believe that libtrixi is much more than just a library that provides some glue code to allow accessing a tool from another language. As noted also on the "Summary" and "Statement of need" sections, there are expecially two areas where we believe that libtrixi clearly meets the requirement for "substantial scholarly effort":

First, there is the technical challenge of being able to use Julia code from another language in the first place. To the best of our knowledge, there exist no similar efforts yet for making the capabilities of a complex Julia package such as Trixi.jl available via a C/Fortran interface. Questions such has how to pass type information between the languages, how to call arbitrary functions, and how to handle the differences between C and Fortran are among those we need to address.

Second, since the Julia programming language is much more flexible than eg C or Fortran regarding its type system, one cannot simply pass data from a main program to Trixi.jl through a C interface (or vice versa) without losing some information and/or flexibility. One major part of libtrixi is thus the Julia package LibTrixi.jl, which aims to encapsulate some of the type complexity of Julia while retaining as much flexibility as possible.

Some of these efforts have not been described in detail in the JOSS submission due to the word limit, but we would be happy to elaborate here (or elsewhere) if that were helpful to come to a (positive) review decision.

cc @bgeihe @gregorgassner

@sloede
Copy link

sloede commented Oct 13, 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:

Flux: Elegant machine learning with Julia
Submitting author: @MikeInnes
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @ysimillides
Similarity score: 0.8026

Optim: A mathematical optimization package for Julia
Submitting author: @anriseth
Handling editor: @labarba (Retired)
Reviewers: @ahwillia
Similarity score: 0.8019

Gridap: An extensible Finite Element toolbox in Julia
Submitting author: @fverdugo
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @PetrKryslUCSD, @TeroFrondelius, @KristofferC
Similarity score: 0.8007

gtk-fortran: a GTK+ binding to build Graphical User Interfaces in Fortran
Submitting author: @vmagnin
Handling editor: @Kevin-Mattheus-Moerman (Active)
Reviewers: @johnsamuelwrites, @sundmanbo
Similarity score: 0.7993

VTUFileHandler: A VTU library in the Julia language that implements an algebra for basic mathematical operations on VTU data
Submitting author: @baxmittens
Handling editor: @jbytecode (Active)
Reviewers: @dmbates, @mkitti
Similarity score: 0.7986

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

@sloede
Copy link

sloede commented Oct 13, 2023

@danielskatz We propose the following scientists as potential reviewers:

  • Samuel Omlin (CSCS/ETHZ, @omlins)
  • Ludovic Räss (Uni Lausanne, @luraess)
  • Jürgen Fuhrmann (WIAS Berlin, @j-fu)

@danielskatz
Copy link

👋 @sloede - I'm sorry to say that after discussion amongst the JOSS editors, we have decided not to review this submission, in part because it does not meet our substantial scholarly effort criterion.

Specific feedback includes

  • Seems to be too small and immature. It also has a short development history.
  • C/C++/Fortran codes in Julia packages are hosted as precompiled binaries as separate packages. This software can not be installed like many canonical Julia packages.

Please see https://joss.readthedocs.io/en/latest/submitting.html#other-venues-for-reviewing-and-publishing-software-packages for other suggestions for how you might receive credit for your work.

We would also be happy to reconsider a future version of the software that addresses these issues (effort, time, maturity, structure).

@danielskatz
Copy link

@editorialbot reject

@editorialbot
Copy link
Collaborator Author

Paper rejected.

@sloede
Copy link

sloede commented Nov 2, 2023

@danielskatz Please excuse my long delay in responding!

👋 @sloede - I'm sorry to say that after discussion amongst the JOSS editors, we have decided not to review this submission, in part because it does not meet our substantial scholarly effort criterion.

Specific feedback includes

* Seems to be too small and immature. It also has a short development history.

* C/C++/Fortran codes in Julia packages are hosted as precompiled binaries as separate packages. This software can not be installed like many canonical Julia packages.

Please see https://joss.readthedocs.io/en/latest/submitting.html#other-venues-for-reviewing-and-publishing-software-packages for other suggestions for how you might receive credit for your work.

We would also be happy to reconsider a future version of the software that addresses these issues (effort, time, maturity, structure).

Thank you very much for the specific feedback, that was very helpful. We are confident that we have understood the reasons for the rejection at this stage. We will make sure to address them before considering a resubmission in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMake Julia Makefile pre-review query-scope Submissions of uncertain scope for JOSS rejected Track: 7 (CSISM) Computer science, Information Science, and Mathematics
Projects
None yet
Development

No branches or pull requests

3 participants