-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
[REVIEW]: SitCoM: Modular framework for the integration of (extreme) events and their impacts in Unity #6458
Comments
Hello humans, 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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
Software report:
Commit count by author:
|
Paper file info: 📄 Wordcount for ✅ The paper includes a |
License info: ✅ License found: |
Review checklist for @ApocalyVecConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @luxaritasConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@luxaritas and @ApocalyVec, I just wanted to give you a gentle reminder of this review. If you have any questions, feel free to ask me. And if you have any suggested changes or improvements to the software, you're welcome to open issues in the source repository or comment here. |
Thanks for the nudge - just took an initial look through this. Before I go any further, I want to raise a concern around JOSS's "substantial scholarly effort" requirement. From my understanding, the work presented here is principally in Assets/_SitCoM, which includes:
I appreciate the value proposition here in being able to create "plug and play" simulations for visualizing extreme events and their impacts. However it doesn't seem to me that the scope as-implemented matches the size JOSS requires. Being mostly generating a UI of switches which conditionally enable objects/object behavior, this feels like it falls under "minor utility package" territory and does not appear to me to be something that would take >= 3 months of time in the context of spinning up a new unity project that needs some UI to do something similar. |
NB: The "real" lines of code here appears to be <1000, which should be flagged according to JOSS guidelines. Most of the auto-identified code is library code |
Thanks a lot for your feedback @luxaritas. @ruedi99ms, @luxaritas and @ApocalyVec, I pause the review for now, until the JOSS editorial team has evaluated whether the submission is within scope. |
@editorialbot commands |
Hello @osorensen, here are the things you can ask me to do:
|
@editorialbot query scope |
Submission flagged for editorial review. |
@editorialbot reject |
Paper rejected. |
Dear author(s), @ruedi99ms, I am sorry to say that JOSS requires certain conditions to be met for this to remain under review, such as the requiremnt to meet our idea for Substantial scholarly effort, which includes but is not limited to the following (see more at previous link):
I am sorry and I hope this does not deter you from choosing JOSS in the future. Best, |
@luxaritas, @ApocalyVec, @osorensen thank you for the valuable contribution you have made even if in this case it did not result in a publication. Sorry again to @ruedi99ms for the less than idea news. |
Submitting author: @ruedi99ms (Niklas Suhre)
Repository: https://github.com/ruedi99ms/SitCoM
Branch with paper.md (empty if default branch):
Version: 1.0
Editor: @osorensen
Reviewers: @luxaritas, @ApocalyVec
Archive: Pending
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@luxaritas & @ApocalyVec, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review.
First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @osorensen know.
✨ Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest ✨
Checklists
📝 Checklist for @ApocalyVec
📝 Checklist for @luxaritas
The text was updated successfully, but these errors were encountered: