bootphon / pygamma-agreement

Gamma Agreement in Python
MIT License
43 stars 8 forks source link

[JOSS] Improve clarity of statement of need section #24

Closed faroit closed 3 years ago

faroit commented 3 years ago

Hi authors, I went once more through the paper and I have three more comments that would be great if those can be addressed in another iteration:

This issue is part of my JOSS review https://github.com/openjournals/joss-reviews/issues/2989

hadware commented 3 years ago

Thanks for the feedback, i'll update the paper to include your remarks.

hadware commented 3 years ago

I updated the paper to include your suggestions. (point 1 and 2 are in the Statements of needs section, point 3 was added as additional explanations under the respective figures). I hope this made the paper clearer.

faroit commented 3 years ago

@hadware thanks a lot. This greatly improves the readability of the paper. I just made a minor comment on a1ccca1 feel free to close this issue (with our without incorporating my proposal)

hadware commented 3 years ago

Thanks for pickup on those again! :) I corrected these small issues in 22e0a82761404c06b7323bc33dabf45fc2256c40 . I'm closing the issue, don't hesitate on reopening it if you feel that some things are still not quite right.