vahid-sb / mitepid_sim Goto Github PK
View Code? Open in Web Editor NEWSimulating stratified model for the spread of COVID19 in a population.
License: GNU General Public License v3.0
Simulating stratified model for the spread of COVID19 in a population.
License: GNU General Public License v3.0
This issue is a part of the JOSS review (openjournals/joss-reviews#2880).
Following the reviewer's checklist:
Do the authors include examples of how to use the software (ideally to solve real-world analysis problems)?
At the moment when I am writting this post neither the functionality nor testing is documented for this project. Therefore I cannot tell what the examples from the directory examples
actually do...
So, right now, my answer to the quoted qestion would be: no.
This issue is a part of the JOSS review (openjournals/joss-reviews#2880).
@vahid-sb
I cannot find any documentation on how to actually use mitepid
... Did I skip it?
The core part of the repository (i.e. your package) should be well documented for the users to know how to use it in their reseach.
This issue is a part of the JOSS review (openjournals/joss-reviews#2880).
@vahid-sb
I am not sure if including your Copyright notice on top of the license text is the correct way to go about it.
Now it states:
Copyright (c) 2019 Vahid Samadi Bokharaie
GNU GENERAL PUBLIC LICENSE
Version 3, 29 June 2007Copyright (C) 2007 Free Software Foundation, Inc. https://fsf.org/
Everyone is permitted to copy and distribute verbatim copies
of this license document, but changing it is not allowed.
So who exactly holds the rights? Vahid Samadi Bokharaie or Free Software Foundation?
Also, please notice that it explicitly says that changing the license document is not allowed...
Could you please look deeper into the case, paste here some info on how to provide a copyright note for a GPL-3 work and adjust accordingly?
This issue is a part of the JOSS review (openjournals/joss-reviews#2880).
Hi @vahid-sb,
I see that the installation is handled by pip
, great!
However, in the README you do not provide versions for the packages your work rely on. You should, if you want to do reproducible research. You cannot guarantee that it will work with every possible combination of versions between all dependencies, right? I assume you did not test them all out. Also, you cannot forsee the future and can't tell if the next minor upgrade in numpy
will not introduce a change in behaviour, thus breaking your code. However, what you can do is to specify the exact versions which you used for your development and testing. And for these you indeed can guarantee the software will work.
So that is what I would ask you to do: please specify exact versions both in the README but also in the setup.py (so that users always get the same versions of dependencies during the installation).
This issue is a part of the JOSS review (openjournals/joss-reviews#2880).
Following the review checklist for the repository's documentation:
A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
Could you please include this information? I see that, in principle, one could extract it from the "Introduction" section in the README but a statement of need is more general and high-level. As far as I am concerned it would be OK just to add one more short section to that file.
This issue is a part of the JOSS review (openjournals/joss-reviews#2880).
Hi @vahid-sb ,
One of the tick boxes in our revision form asks if the functional claims of the software have been confirmed. Another one: "Are there automated tests or manual steps described so that the functionality of the software can be verified?"
I could not find any CI nor a walkthrough on how to carry out the simulations, however, I noticed a lot of scripts in the examples
dir...
Could you please include this information? It would be really nice if you add a Continous Integration mechanisms with unit tests / simulations. Preferably, GitHub Actions, so that you keep everything related to the project in one place (however, I would not mind other CI solutions, if you prefer any).
====
EDIT:
I notice there are quite a few claims about the functionality of the package you listed in the README; all of these should be verifiable.
=====
EDIT2:
Later in the README is says:
There are a few scripts in the examples folder in the repository which can be used as examples of what the code can do. To see samples of the plots generated by the code, you can look at this manuscript or this one . All the plots in these two manuscripts are generated using MiTepid_sim.
To have such description is unfortunately, not sufficient. You need to make it transparent. When a paper passes JOSS revision it will get a digital object identifier and will essentially get "immortalised". Therefore, the repository should be as much self-contained as possible, meaning code runs, tests, examples etc. well-documented here.
This issue is a part of the JOSS review (openjournals/joss-reviews#2880).
Hi @vahid-sb,
I compiled and looked closer at your paper.md.
In my opinion it requires major revisions before we can proceed with the revisions checks.
My checklist points out:
Summary: Has a clear description of the high-level functionality and purpose of the software for a diverse, non-specialist audience been provided?
A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
State of the field: Do the authors describe how this software compares to other commonly-used packages?
Quality of writing: Is the paper well written (i.e., it does not require editing for structure, language, or writing quality)?
References: Is the list of references complete, and is everything cited appropriately that should be cited (e.g., papers, datasets, software)? Do references in the text use the proper citation syntax?
The points I could raise now:
we can stratify the into various relevant groups, for example, age-groups.
, There are a few scripts
, what the code can do
, you can look
and the excessive use of word quite
.I hope these bullet points will guide you in the right direction.
Please let me know once you push version 2.0 of the publication.
This issue is a part of the JOSS review (openjournals/joss-reviews#2880).
Hi @vahid-sb,
From the reviewer's list:
Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support
Please add these explicite. Also, it would be really good if you add a CODE_OF_CONDUCT for the project too :)
A declarative, efficient, and flexible JavaScript library for building user interfaces.
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. ๐๐๐
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google โค๏ธ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.