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]: A C++ template library for polynomials algebra over discrete euclidean domains #5942

Closed
editorialbot opened this issue Oct 13, 2023 · 39 comments
Assignees
Labels
C++ Makefile pre-review TeX Track: 7 (CSISM) Computer science, Information Science, and Mathematics

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Oct 13, 2023

Submitting author: @JeWaVe (regis portalez)
Repository: https://github.com/aerobus-open-source/aerobus
Branch with paper.md (empty if default branch):
Version: v1.1
Editor: @gkthiruvathukal
Reviewers: @mmoelle1, @lucaferranti, @pitsianis
Managing EiC: Daniel S. Katz

Status

status

Status badge code:

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

Author instructions

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

@JeWaVe 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 (306.9 files/s, 109753.1 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
C/C++ Header                     1            358             91           1494
C++                              2            129             42            976
Markdown                         2            110              0            565
JSON                             3              0              0             85
YAML                             1              4              4             31
TeX                              1              3              0             27
make                             1              4              0             11
-------------------------------------------------------------------------------
SUM:                            11            608            137           3189
-------------------------------------------------------------------------------


gitinspector failed to run statistical information for the repository

@editorialbot
Copy link
Collaborator Author

Wordcount for paper.md is 2200

@editorialbot
Copy link
Collaborator Author

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

OK DOIs

- 10.1007/978-3-319-06486-4_7 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:

Universal Numbers Library: Multi-format Variable Precision Arithmetic Library
Submitting author: @Ravenwater
Handling editor: @danielskatz (Active)
Reviewers: @mmoelle1, @mlxd, @yboettcher
Similarity score: 0.8229

Castor: A C++ library to code "à la Matlab"
Submitting author: @matthieuaussal
Handling editor: @jbytecode (Active)
Reviewers: @mkitti, @pitsianis
Similarity score: 0.8150

cppduals: a nestable vectorized templated dual number library for C++11
Submitting author: @tesch1
Handling editor: @danielskatz (Active)
Reviewers: @StewMH, @mdavezac
Similarity score: 0.8120

ChebTools: C++11 (and Python) tools for working with Chebyshev expansions
Submitting author: @ianhbell
Handling editor: @labarba (Retired)
Reviewers: @drvinceknight
Similarity score: 0.8074

Cadabra2: computer algebra for field theory revisited
Submitting author: @kpeeters
Handling editor: @danielskatz (Active)
Reviewers: @StewMH, @henrik227
Similarity score: 0.8073

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

@gkthiruvathukal - would you be willing to edit this submission?

@danielskatz
Copy link

@editorialbot invite @gkthiruvathukal as editor

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@gkthiruvathukal
Copy link

@danielskatz Yes. Will assign self as editor!

@gkthiruvathukal
Copy link

@editorialbot assign me as editor

@editorialbot
Copy link
Collaborator Author

Assigned! @gkthiruvathukal is now the editor

@JeWaVe
Copy link

JeWaVe commented Oct 17, 2023

@editorialbot generate pdf

I pushed some modifications in both paper and bibligraphy.

@editorialbot
Copy link
Collaborator Author

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

@gkthiruvathukal
Copy link

@drvinceknight Are you willing to help review this JOSS submission?

@gkthiruvathukal
Copy link

@mmoelle1 Are you willing to help review this JOSS submission?

@gkthiruvathukal
Copy link

Once I have two reviewers, review will begin.

@gkthiruvathukal
Copy link

@JeWaVe As we await the reviewers, I notice that this C++ project is a bit lacking when it comes to the build process. The Makefile doesn't appear to be portable. You should consider using a Makefile generator (e.g. autoconf) or a more modern build system. I personally use CMake in my projects.

@JeWaVe
Copy link

JeWaVe commented Oct 23, 2023 via email

@mmoelle1
Copy link

Yes, @gkthiruvathukal I am willing to review this JOSS submission

@JeWaVe
Copy link

JeWaVe commented Oct 29, 2023

Hi @gkthiruvathukal I just added a cmakelists.txt and updated the readme.
In addition, I added some documentation (not complete at all) and updated the readme (main branch)

@JeWaVe
Copy link

JeWaVe commented Nov 27, 2023

Hi. It's been a month with no news. Where are we in the reviewing process ? (I do not publish that often so I don't know if it's normal or not).

@gkthiruvathukal
Copy link

@JeWaVe I am sorry for the delay. Things have been uncharacteristically busy in my life but I will try to make some forward progress here.

@gkthiruvathukal
Copy link

@editorialbot add @mmoelle1 as reviewer

@editorialbot
Copy link
Collaborator Author

@mmoelle1 added to the reviewers list!

@gkthiruvathukal
Copy link

@henrik227 Are you willing to help review this JOSS submission?

@gkthiruvathukal
Copy link

@pitsianis Are you willing to help review this JOSS submission?

@gkthiruvathukal
Copy link

@mkitti Are you willing to review this JOSS submission?

@JeWaVe
Copy link

JeWaVe commented Jan 10, 2024

Hi everyone,

I just added a major update with implementation of galois fields (GF(p, n) for all p < 1000 and n < 20).
Everything is in the main branch, I hope I didn't break anything here.

@mkitti
Copy link

mkitti commented Jan 10, 2024

@gkthiruvathukal I'm still finishing another review and otherwise busy. I might be able to join later if needed once that concludes. I have discussed this with @lucaferranti, and he is interested and is capable as a reviewer.

@lucaferranti
Copy link
Member

@gkthiruvathukal , yes I can review this

@pitsianis
Copy link

My apologies for replying late, I will be happy to assist if needed.

@gkthiruvathukal
Copy link

@editorialbot add @lucaferranti as reviewer

@editorialbot
Copy link
Collaborator Author

@lucaferranti added to the reviewers list!

@gkthiruvathukal
Copy link

@editorialbot add @pitsianis as reviewer

@editorialbot
Copy link
Collaborator Author

@pitsianis added to the reviewers list!

@gkthiruvathukal
Copy link

@lucaferranti and @pitsianis Thank you for being willing to review this submission. Now I can start the review...

@gkthiruvathukal
Copy link

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ Makefile pre-review TeX Track: 7 (CSISM) Computer science, Information Science, and Mathematics
Projects
None yet
Development

No branches or pull requests

8 participants