Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign up[REVIEW]: NodeLab: A MATLAB package for meshfree node-generation and adaptive refinement #1173
Comments
whedon
assigned
kyleniemeyer
Jan 14, 2019
whedon
added
the
review
label
Jan 14, 2019
whedon
referenced this issue
Jan 14, 2019
Closed
[PRE REVIEW]: NodeLab: A MATLAB package for meshfree node-generation and adaptive refinement #1152
This comment has been minimized.
This comment has been minimized.
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @Kevin-Mattheus-Moerman, it looks like you're currently assigned as the reviewer for this paper If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews To fix this do the following two things:
For a list of things I can do to help you, just type:
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Jan 14, 2019
|
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Jan 29, 2019
@whedon commands |
This comment has been minimized.
This comment has been minimized.
Here are some things you can ask me to do:
|
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Jan 29, 2019
@kyleniemeyer: Is there anything required from my side at this point? |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Jan 29, 2019
@pankajkmishra not at the moment—the reviewers will report any issues requiring your attention here or in the software repository as appropriate. |
This comment has been minimized.
This comment has been minimized.
I should be able to complete the full review by next Monday. It has been a busy couple of weeks. @pankajkmishra @kyleniemeyer |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Feb 18, 2019
@@kyleniemeyer: Is it allowed to update the repository while it is under review? |
This comment has been minimized.
This comment has been minimized.
I'll finalize the review by next Monday. Apologies for the delay. |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Feb 19, 2019
@pankajkmishra Yes, but please let us know here when you update it and what changes were made; it sounds like the reviewers may not have started with the current version, so should be fine. |
This comment has been minimized.
This comment has been minimized.
@kyleniemeyer Should I upload a list of my suggestions separately or just continue discussion with the author here as part of the comment thread ? @pankajkmishra I also noticed that there is no official release for NodeLab yet. So it may be appropriate to update the versioning info first and then tag the commit as a checkpoint for the review. Thoughts @kyleniemeyer ? |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Feb 26, 2019
@vijaysm feel free to continue the discussion here; you can submit a list of suggestions as a single comment and we can go from there. Regarding the official release, I'm fine if @pankajkmishra waits until after making any changes as part of this review process, and then makes an official release associated with the version accepted here. |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Feb 27, 2019
Thanks @vijaysm! I'll look forward to your suggestions. @kyleniemeyer - That would be convenient! Thanks. |
This comment has been minimized.
This comment has been minimized.
@pankajkmishra Here are some comments on the paper:
|
This comment has been minimized.
This comment has been minimized.
@whedon check references |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
I've created a pull request (pankajkmishra/NodeLab#2) to enhance the demos/documentation. I recommend clarifying the required input arguments and perhaps to visualize them as I suggest. |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Mar 8, 2019
Thanks @Kevin-Mattheus-Moerman! I'll get back to you after implementing your suggestions. |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Mar 8, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@pankajkmishra great. Can you also work on adding those DOI's? Let me know if you have questions. |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Mar 8, 2019
@whedon check references |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Aug 2, 2019
•
@kyleniemeyer I have updated the paper showing better a clarity for the value addition. |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Aug 2, 2019
Thanks @pankajkmishra. @vijaysm, do you recommend acceptance based on the changes made? |
This comment has been minimized.
This comment has been minimized.
@pankajkmishra Thanks for the description update. @kyleniemeyer While the updated paper makes the application space clearer than before, I would like to reiterate that the work on NodeLab still remains incremental. Node generation support for 3-D volumes or surfaces would certainly compel me to accept the paper. Also, other open-source Matlab packages by Per-Olof Persson (Distmesh [1]) and by Darren Engwirda (Mesh2D [2]), already do provide adaptive node generation and delaunay triangulation based on user-defined distance functions as well for 2-D domains. Additionally, I think some NodeLab examples showing adaptive refinement with integration to a mesh-free solver would have added value to this contribution with a practical use-case. As it stands, I am taking the decision to reject the paper submission. @pankajkmishra, please consider updating examples and adding support for 3-D domains to differentiate this work from other existing open-source alternatives. I would be happy to discuss my comments further if you have any questions. [1] http://persson.berkeley.edu/distmesh/ |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Aug 4, 2019
Hi @vijaysm, thanks for your input. I should point out that JOSS does not reject submissions for novelty, originality, or impact; instead, we only reject when the authors are unwilling to improve the software to meet our standards. Since both you and @Kevin-Mattheus-Moerman have checked off all the review items, and it doesn't seem like there are any remaining items of that nature, I am going to move this into the final acceptance stages. @pankajkmishra at this point please archive the software repository (e.g., in Zenodo) and provide the DOI here. I'm going to do a final review/edit of the paper itself. |
This comment has been minimized.
This comment has been minimized.
@kyleniemeyer Fair enough. In that case, the description update from @pankajkmishra was all that was needed to make the submission complete for me. |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Aug 6, 2019
Hey @kyleniemeyer , thanks! I'll do it ASAP. |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Aug 6, 2019
@kyleniemeyer following is the Zenodo DOI |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Aug 12, 2019
@whedon set 10.5281/zenodo.3361734 as archive |
This comment has been minimized.
This comment has been minimized.
OK. 10.5281/zenodo.3361734 is the archive. |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Aug 12, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Aug 12, 2019
Hi @pankajkmishra , just a few final edits needed on the paper:
There are a few more places in the paper that should be edited:
Aside from that, the first paragraph should start with a statement of need, meant for a general reader. I think you could move your sentence about the applications here, and do some revision to make it work. |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Aug 15, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Aug 15, 2019
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Aug 15, 2019
@openjournals/joss-eics ok, this is ready for acceptance from my point of view |
This comment has been minimized.
This comment has been minimized.
pankajkmishra
commented
Aug 15, 2019
Thanks @kyleniemeyer , and everyone :) |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 15, 2019
@whedon accept |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
Check final proof If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#903, then you can now move forward with accepting the submission by compiling again with the flag
|
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 15, 2019
Thanks to @Kevin-Mattheus-Moerman and @vijaysm for reviewing and @kyleniemeyer for editing |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 15, 2019
@whedon accept deposit=true |
whedon
added
the
accepted
label
Aug 15, 2019
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
Here's what you must now do:
Any issues? notify your editorial technical team... |
danielskatz
closed this
Aug 15, 2019
This comment has been minimized.
This comment has been minimized.
If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
whedon commentedJan 14, 2019
•
edited
Submitting author: @pankajkmishra (Pankaj K Mishra)
Repository: https://github.com/pankajkmishra/NodeLab
Version: v1.0
Editor: @kyleniemeyer
Reviewer: @Kevin-Mattheus-Moerman, @vijaysm
Archive: 10.5281/zenodo.3361734
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) 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
@Kevin-Mattheus-Moerman & @vijaysm, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @kyleniemeyer know.
Review checklist for @Kevin-Mattheus-Moerman
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?Review checklist for @vijaysm
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?