Skip to content

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

Closed
5 tasks done
dominiquesydow opened this issue Apr 3, 2022 · 3 comments
Closed
5 tasks done

JOSS review on software paper #201

dominiquesydow opened this issue Apr 3, 2022 · 3 comments

Comments

@dominiquesydow
Copy link

dominiquesydow commented Apr 3, 2022

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)

  • Could you please elaborate a bit more on the gap that GEMMI is filling?
    • Examples of overlaps and trade-offs are nicely stated - which choices were made for GEMMI and why?
    • Are there examples of functionalities that the other packages do not have?
  • If applicable (going hand-in-hand with the previous point), could you state in which cases a user might want to use GEMMI vs. CCTBX vs. Clipper? As someone who has not used any of these software packages, that would be my first question to deciding on which software to spend my time on.
  • Does it make sense to mention coot alongside software that has (at least a subset of similar) functionalities?
  • UglyMol is mentioned in the "Summary" as an application that uses GEMMI if I understand correctly. Would make sense to add it also the list at the end of the "Statement of need" alongside autoBUSTER etc.?
  • References: If applicable, could you please add DOIs where missing?

Link to the review: openjournals/joss-reviews#4200

@dominiquesydow
Copy link
Author

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

@wojdyr
Copy link
Member

wojdyr commented Apr 4, 2022

Hi @dominiquesydow

thanks for taking the time to review this paper.

* [ ]  Could you please elaborate a bit more on the gap that GEMMI is filling?
  
  * Examples of overlaps and trade-offs are nicely stated - which choices were made for GEMMI and why?
  * Are there examples of functionalities that the other packages do _not_ have?

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.

* [ ]  If applicable (going hand-in-hand with the previous point), could you state in which cases a user might want to use GEMMI vs. CCTBX vs. Clipper? As someone who has not used any of these software packages, that would be my first question to deciding on which software to spend my time on.

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.
Heavy users will often need custom features or changes in existing features. So they will need to frequently communicate with the project they use. And because they often work in a group doing method development for crystallography, and this groups is usually invested in one of the libraries, they go with what their colleagues use.

* [ ]  Does it make sense to mention coot alongside software that has (at least a subset of similar) functionalities?

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).

* [ ]  UglyMol is mentioned in the "Summary" as an application that uses GEMMI if I understand correctly.  Would make sense to add it also the list at the end of the "Statement of need" alongside autoBUSTER etc.?

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.

* [ ]  References: If applicable, could you please add DOIs where missing?

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.

@wojdyr
Copy link
Member

wojdyr commented Apr 24, 2022

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.

@wojdyr wojdyr closed this as completed Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants