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]: diman: A Clojure package for dimensional analysis #3735
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @oyvinht, @khinsen it looks like you're currently assigned to review this paper 🎉. Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post. ⭐ 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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
Wordcount for |
|
|
👋 @oyvinht, please update us on how your review is going (this is an automated reminder). |
👋 @khinsen, please update us on how your review is going (this is an automated reminder). |
Review update: I am looking at the code and playing with it. Interacting with the author via issues on the code repository. |
@lungsi @mjsottile Here comes my review! The software described by this submission can be characterized as a highly specialized computer algebra system, in that is performs symbolic rather then the more common (in science) numeric computations. The field of application is dimensional analysis, which is a very useful but so far underappreciated tool in scientific computing. The software does exactly what the documentation and the paper promise, and the documentation is sufficient not only to learn how to use the software, but even to learn about the techniques that the software implements. The unchecked points in my reviewer checklist are all related, and ultimately due to the fact that this software package does not exactly fit JOSS' ideas of what it wants to publish. However, I believe that software packages like this one should be published, and I don't see any particular reason why they should not be published in JOSS. So in the end, this paper requires an editorial policy decision. The first point of hesitation is "Substantial scholarly effort". Looking at the list of criteria, this package is at the edge. It's relatively small, and was developed by a single person in a relatively short timeframe. But this does not mean it is trivial, it is merely very focused: a small tool for a very particular use case. A related point is the absence of community guidelines. It would be trivial for the author to add some boilerplate along the lines of "contribute via GitHub features". But the real point is that this package doesn't have a community and doesn't need one. It's small enough to be understood by a single person, even someone else than the author. I wouldn't mind using it for a research project even if I knew for sure there were no support at all. But it is nevertheless a useful contribution to science, because it is much less effort to use this package than to develop my own. In fact, I believe that packages such as this should be published because they make underappreciated techniques more accessible. The final unchecked point is "state of the field". I am now aware of any similar package, and assuming that @lungsi doesn't either, he/she could simply add a sentence stating this. But that misses the point that the main reason for publishing this package is the lack of any other tools that implement the same techniques. |
👋 @oyvinht Let me know if you are blocked on anything with respect to this review that I can assist with. |
@whedon @lungsi @mjsottile Here is my review: Background Relevance Code guidelines
Summary Points 3 and 4 above are more about "taste", but it is my belief that most Clojure users would consider the code more idiomatic if they were adhered to. |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
@whedon commands |
Here are some things you can ask me to do:
|
@whedon list reviewers |
Here's the current list of reviewers: https://bit.ly/joss-reviewers |
OK, @oyvinht is now a reviewer |
@whedon set 10.5281/zenodo.5837630 as archive |
OK. 10.5281/zenodo.5837630 is the archive. |
@mjsottile Thank you very much. I have made the changes as suggested.
|
@whedon generate pdf |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2883 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2883, then you can now move forward with accepting the submission by compiling again with the flag
|
@lungsi - I've suggested some small changes in neuralgraphs/diman#10 - please merge this or let me know what you disagree with. |
I have merged it. Thanks @danielskatz |
@danielskatz The last two items have now been checked off. |
@danielskatz Done! Nice to see this published! |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2887 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2887, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true |
|
🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦 |
🚨🚨🚨 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 to @lungsi (B. Lungsi Sharma)!! And thanks to @oyvinht and @khinsen for reviewing, and to @mjsottile for editing! |
🎉🎉🎉 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:
|
Thank you very much (reviewers: @oyvinht and @khinsen, and editors: @mjsottile and @danielskatz). Comments by @oyvinht and @khinsen were very helpful, I learned some new stuffs. I really appreciated the generous comments from @khinsen, it was very encouraging. I would also like to point out that @mjsottile 's feedback greatly improved the manuscript. Overall, my experience with JOSS has been highly rewarding (regardless of the paper being accepted, although this helps :) ). |
Submitting author: @lungsi (B. Lungsi Sharma)
Repository: https://github.com/neuralgraphs/diman
Version: v1.1.0
Editor: @mjsottile
Reviewers: @oyvinht, @khinsen
Archive: 10.5281/zenodo.5837630
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
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
@oyvinht & @khinsen, 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.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @mjsottile 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 ✨
Review checklist for @oyvinht
✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @khinsen
✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: