-
Notifications
You must be signed in to change notification settings - Fork 47
JOSS review on software paper #201
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
Comments
Adding to the list above, at the moment the paper describes GEMMI nicely as a toolbox for working with crystallographic data. I think it would be beneficial to add a paragraph that shows examples of how this toolbox could be put together for example use cases (e.g. common tasks in a crystallographer's work life). This goes very much in hand with what I felt was missing when I went through the documentation and described in this issue: #202 |
thanks for taking the time to review this paper.
There are many smaller and larger differences, but I couldn't find a representative example that would do justice to all the compared packages. Cctbx has order of magnitude more man-years of development and it doesn't have major gaps, but for sure there are small features in gemmi and clipper that are not in cctbx.
There is no easy answer to this. If the user needs only a small subset of functionality (say, reading a CIF file) they could compare how it can be in different packages, as well as how easy it's to find documentation and install different libraries.
Coot is using Clipper for most of the similar functionalities. (Coot development started in York, where Clipper is being developed). But thinking about it, I should mention MMDB (another library used by Coot).
The list of projects is the Statement of need is far from complete. Github show that 58 repositories depend on gemmi, so I didn't try to list all of them.
Yes, I'll update the paper soon. I'll think about examples that could be added. This would need to be a new section in the paper. Thanks for the review. |
Thanks for thoughtful reading the paper. Comparison with other packages is a sensitive thing and it would be difficult to make for me, because I'm obviously biased. I'll rather wait for someone else to write it. But I understand such comparisons are useful. Recently I was trying to make sense of the landscape in molecular dynamics packages, but I haven't found any comparisons in documentations of these packages. I added DOIs to all references (that have DOIs) and updated the paper. |
Hi @wojdyr,
I just read through the manuscript - it is well-written and has a clear and concise summary, very much appreciated.
I have a few comments on the "Statement of need" section, referring to this paper version: openjournals/joss-reviews#4200 (comment)
Link to the review: openjournals/joss-reviews#4200
The text was updated successfully, but these errors were encountered: