-
-
Notifications
You must be signed in to change notification settings - Fork 36
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]: CusToM: a Matlab toolbox for musculoskeletal simulation #927
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @modenaxe, it looks like you're currently assigned as the reviewer for this paper 🎉. ⭐ Important ⭐ 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:
|
|
@modenaxe, @demotu this is where the review will take place. You each have a set of check boxes at the top of this issue to guide you through the review process. Please let me know if you have any questions or check the review guidelines. Thanks! 🚀 🤖 |
@demotu We will get started here already but it is fine if you join the review process in October. Thanks. |
@anmuller this is where the review process takes place so please keep an eye out for notifications from this issue. Here are some initial comments/questions I have:
|
@Kevin-Mattheus-Moerman, I am sorry but it is not compatible with Octave. Thank you for your recommendations about community guidelines, I am going to work it. |
I just started checking the paper and the material. It's the first time I review for JOSS, so please be patient with me @Kevin-Mattheus-Moerman if I involve you at times :) General checks
Functionality Documentation
Software paper
|
@modenaxe thank you for your review. What are the errors that appeared when you run 'GenerateParameters.mlapp'? |
I have posted them as an issue in the repository, let me know if you want more details. |
@nicoguaro would you be interested in joining this review process? I can add you as a reviewer if so. |
@anmuller can you give an update as to how you are getting on in terms of replying to the issues raised? |
@demotu how are you getting on? Could you give an indication as to when you will complete the review process? Thanks. |
I am updating my comments after testing the revised toolbox. @anmuller my comments follow exactly the guidelines and checkboxes above. @Kevin-Mattheus-Moerman this is currently my complete review, excluding the issues that I will post in the repository directly. General checks
Functionality Documentation
Software paper
|
@Kevin-Mattheus-Moerman, I think I can. I have a question, though, is MATLAB necessary for the software or can it be used with Octave? If MATLAB is required, what is the minimum version needed? Also, from just checking the README in the repo I can't find installation instructions for the package. Or a simple example for "starters". I also suggest to format the README to take advantage of Markdown. |
@whedon add @nicoguaro as reviewer |
OK, @nicoguaro is now a reviewer |
@nicoguaro there is now also a checklist for you at the top of this issue. Yes I believe MATLAB is required. @anmuller can you comment on what version is required? |
It was implemented and tested with Matlab2018a. Also the Symbolic Math, the Optimization and the Parallel Computing toolboxes are necessary |
@anmuller I have access to Matlab2017a, is that OK. I can't find installation instructions and a test script to check that things work. |
@nicoguaro Maybe you can use Matlab2017a but I can't ensure you it works. |
@anmuller, I consider that both the installation instructions and example should be in the repo. Hopefully, in the README. |
@whedon set v1.1.0 as version |
OK. v1.1.0 is the version. |
@anmuller — The DOI link on your reference list for Featherstone (2008) points to a book titled "Robot Dynamics Algorithms." Did you get the title wrong? Also, it looks like the book does not have 272 pages—and please use a ~ to get the spacing right: (p.~272). |
Finally, journal titles are capitalized in English-language publication. You're mixing English and French conventions. Please capitalize throughout. |
@labarba correct there was an issue with Linux and this limitation was added to the descriptions/documentation. Rather than removing the reviewer I thought it best to leave these statements for the record. |
@whedon generate pdf |
|
@whedon generate pdf |
|
@labarba thanks for your comments. |
Great. Glad I caught that reference oops. I do have a couple more editorial fixes:
Par. 1:
Par. 2:
Par. 3:
Finally, a gentle suggestion to strip out some of the unnecessary passive voice; e.g., "A user interface has been developed…" >> by whom? |
@whedon generate pdf |
|
@labarba thanks again for the comments and for the advices. The changes were done. |
@whedon accept |
|
Check final proof 👉 openjournals/joss-papers#447 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#447, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true |
|
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨 Here's what you must now do:
Any issues? notify your editorial technical team... |
Congratulations, @anmuller, your paper is published in JOSS! 🎉 Thank you, gracias, merci to your handling editor, @Kevin-Mattheus-Moerman, and your reviewers, @modenaxe, @demotu, @nicoguaro 🙏 |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 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:
|
Thanks very much @labarba, @Kevin-Mattheus-Moerman, @modenaxe, @demotu, @nicoguaro for all your advices and the time spent on this review. |
Submitting author: @anmuller (Antoine Muller)
Repository: https://github.com/anmuller/CusToM.git
Version: v1.1.0
Editor: @Kevin-Mattheus-Moerman
Reviewers: @modenaxe, @demotu, @nicoguaro
Archive: 10.5281/zenodo.2543645
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
@modenaxe & @demotu, 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 @Kevin-Mattheus-Moerman know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @modenaxe
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 @demotu
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 @nicoguaro
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?The text was updated successfully, but these errors were encountered: