Skip to content

[PRE REVIEW]: Decomposing Gaifman Structures #7687

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

Closed
editorialbot opened this issue Jan 17, 2025 · 23 comments
Closed

[PRE REVIEW]: Decomposing Gaifman Structures #7687

editorialbot opened this issue Jan 17, 2025 · 23 comments
Assignees
Labels
pre-review Python query-scope Submissions of uncertain scope for JOSS TeX Track: 5 (DSAIS) Data Science, Artificial Intelligence, and Machine Learning withdrawn

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Jan 17, 2025

Submitting author: @balqui (José Luis Balcázar)
Repository: https://github.com/balqui/degais/
Branch with paper.md (empty if default branch):
Version: V1.4
Editor: @matthewfeickert
Reviewers: Pending
Managing EiC: Chris Vernon

Status

status

Status badge code:

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

Author instructions

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

@balqui 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: 5 (DSAIS) Data Science, Artificial Intelligence, and Machine Learning labels Jan 17, 2025
@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

Checking the BibTeX entries failed with the following error:

Failed to parse BibTeX on value "url" (NAME) [#<BibTeX::Bibliography data=[6]>, "@", #<BibTeX::Entry >, {:author=>["Andrzej Ehrenfeucht and Tero Harju and Grzegorz Rozenberg"], :title=>["The Theory of 2-Structures - {A} Framework for Decomposition and Transformation of Graphs"], :publisher=>["World Scientific"], :year=>["1999"]}, ",", "%"]

@editorialbot
Copy link
Collaborator Author

Software report:

github.com/AlDanial/cloc v 1.98  T=0.29 s (217.6 files/s, 511833.0 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Text                             6             39              0          76076
TableGen                        27              0              0          70763
Python                          25            418            912           1644
Markdown                         3            105             10            398
TeX                              1             16              1             95
YAML                             1              1              4             23
TOML                             1              3              0             21
-------------------------------------------------------------------------------
SUM:                            64            582            927         149020
-------------------------------------------------------------------------------

Commit count by author:

    95	balqui
    52	José Luis Balcázar, balqui at GitHub
     7	Balqui

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 1433

✅ The paper includes a Statement of need section

@editorialbot
Copy link
Collaborator Author

License info:

✅ License found: MIT License (Valid open source OSI approved license)

@editorialbot
Copy link
Collaborator Author

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

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

PyDGN: a Python Library for Flexible and Reproducible Research on Deep Learning for Graphs
Submitting author: @diningphil
Handling editor: @arfon (Active)
Reviewers: @idoby, @sepandhaghighi
Similarity score: 0.6651

CORA and LOGIGRAM: A Duo of Python Packages for Combinational Regularity Analysis (CORA)
Submitting author: @ZuzanaSebb
Handling editor: @vissarion (Active)
Reviewers: @Naeemkh, @Arrondissement5etDemi
Similarity score: 0.6536

DAFSA: a Python library for Deterministic Acyclic Finite State Automata
Submitting author: @tresoldi
Handling editor: @gkthiruvathukal (Active)
Reviewers: @geoffbacon, @desilinguist
Similarity score: 0.6478

TLViz: Visualising and analysing tensor decomposition models with Python
Submitting author: @marieroald
Handling editor: @faroit (Active)
Reviewers: @sara-02, @yiitozer
Similarity score: 0.6469

Clustergram: Visualization and diagnostics for cluster analysis
Submitting author: @martinfleis
Handling editor: @csoneson (Active)
Reviewers: @csadorf, @gagolews
Similarity score: 0.6457

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

@crvernon
Copy link

👋 @balqui

Could you explain this statement a bit? Why was the refactoring done and is any of the original code used directly in your work? Thanks.

Author: Jose Luis Balcazar, ORCID 0000-0003-4248-4528, relying in large parts on the functionality of code by Marie Ely Piceno, although fully rewritten.

@balqui
Copy link

balqui commented Jan 18, 2025 via email

@crvernon
Copy link

crvernon commented Feb 5, 2025

@editorialbot invite @matthewfeickert as editor

@balqui please reduce the word count of your paper to at or lower than 1000 words. Thank you.

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@matthewfeickert
Copy link
Member

@editorialbot assign @matthewfeickert as editor

@editorialbot
Copy link
Collaborator Author

Assigned! @matthewfeickert is now the editor

@matthewfeickert
Copy link
Member

matthewfeickert commented Feb 5, 2025

@balqui thanks for your submission to JOSS. Before I request any reviewer time to be spent on the submission though I'm going to ask you to reread the Submitting a paper to JOSS docs in detail as there are multiple problems with the submission that would result in it being automatically declined without large amounts of revisions by reviewers.

It might be helpful for you to read the Review Criteria docs but at a glance your submission has:

  • No documentation (a doc.md is insufficient)
  • Installation instructions are not prominent
  • The directory structure for degais is wrong and does not package degais properly as a Python package.
    • Specifically it does not correctly follow a src/ directory layout of src/<package name>/ which is required to properly lay out the package in a distribution. Installation of degais as is incorrectly installs the Python source files directly under the virtual environment's site-packages/ directory, not under site-packages/degais/ (which doesn't exist).
  • Only one example which is just text, no code, and so insufficient
  • No API docs
  • No tests that can be run in an automated way with a testing framework
  • No CI

Please address all of these issues within one week (by 2025-02-13) if you would like to continue with a review.

My suggestion given the scope of work that would need to be done is that you instead withdraw your submission for now and resubmit in the future whenever you have had the time to carefully go through and prepare the project. That's totally fine and we would very much welcome a future submission from you if you chose to do so.

@matthewfeickert
Copy link
Member

@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 Feb 6, 2025
@balqui
Copy link

balqui commented Feb 6, 2025

Understood, Matthew and Chris. We will follow the advice of withdrawing and possibly resubmitting as the issues raised require clearly more than one week to solve. Thanks for your help. José Luis Balcázar (@balqui)

@balqui
Copy link

balqui commented Feb 6, 2025

@editorialbot withdraw

@editorialbot
Copy link
Collaborator Author

I'm sorry @balqui, I'm afraid I can't do that. That's something only eics are allowed to do.

@balqui
Copy link

balqui commented Feb 6, 2025

Sorry, I am told that I cannot withdraw through the editorial bot, and I cannot find on the submission page (which to me looks like https://joss.theoj.org/papers/2bc1b254b01c7202572d4d2d1dc07fb7) a way to withdraw. I do see a "paper actions" section with some text in it but no actual action available to perform. How should we do that?

@danielskatz
Copy link

As the error message says, Editors-in-Chief (eics) can do this, as I will now.

@danielskatz
Copy link

@editorialbot withdraw

@editorialbot
Copy link
Collaborator Author

Paper withdrawn.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pre-review Python query-scope Submissions of uncertain scope for JOSS TeX Track: 5 (DSAIS) Data Science, Artificial Intelligence, and Machine Learning withdrawn
Projects
None yet
Development

No branches or pull requests

5 participants