Comments (14)
from joss-reviews.
/ cc @openjournals/joss-reviewers - would anyone be willing to review this submission?
If you would like to review this submission then please comment on this thread so that others know you're doing a review (so as not to duplicate effort). Something as simple as :hand: I am reviewing this
will suffice.
Reviewer instructions
- Please work through the checklist at the start of this issue.
- If you need any further guidance/clarification take a look at the reviewer guidelines here http://joss.theoj.org/about#reviewer_guidelines
- Please make a publication recommendation at the end of your review
Any questions, please ask for help by commenting on this issue!
from joss-reviews.
General checks
all checked
Functionality
all checked
- I confirmed the functionality using code from the test suite
Documentation
some checked
Missing:
- Example Usage: There is no example usage or command to run the software using some of the sample data included. Recommendation: Add a 'Example usage' section with step-by-step commands to run Xenomapper with files from
xenomapper/tests/data
as input and explanation of each problem solved. More than a single example would be great to have examples of some of the multiple features. - Community guidelines: There is no mention on how to contribute, report bugs or get support. Recommendation: Add a 'How to contribute' section with at least a brief statement encouraging posible contributors to help pointing them to open a new github issue. Even better would be to link to a CONTRIBUTING.md file with extended instructions. Add also comments on how to report a problem (opening an issue should be OK) and seek support (ideas: mailing list, contact email, github issue with a specific tag).
Software paper:
all checked
- Authors and statement of needs OK
- All DOIs verified
Nice-to-have improvements:
- License file is included in the repository, but users need to open it to know what is the actual license used. I'd recommend adding a mention in the readme file. Just something like "Licensed under GPL v3" would be useful.
- Tests are included and Travis is configured, so that point is clearly checked. But It would be good if instructions on how to run the tests locally are clearly stated in the README. Actually the command to do it (
python3 -m xenomapper.tests.test_all
) is there, but under theAlternate Installation
section.
Recommendation:
I recommend publication after some minor revisions: usage examples and contributing instructions should be added to the documentation.
from joss-reviews.
I recommend publication after some minor revisions: usage examples and contributing instructions should be added to the documentation.
Many thanks for the review @xuanxu. @genomematt - please let us know when you've addressed @xuanxu's comments and we can move forward.
from joss-reviews.
@xuanxu @afron - updated version addressing review comments pushed. Examples are still a bit sparse but I will continue to add to them. Data access issues with most of the worked examples I already have so I will need to hunt up some more datasets. I really want these to be runnable notebooks. I also added an extra affiliation line to reflect my dual appointment to the paper.md
from joss-reviews.
I've checked both points and I consider this publication-ready now.
A small doubt, @arfon & @genomematt: the archive link in this issue and the "DOI for current release" link in the "How to cite Xenomapper" section both still point to version 1.0.0. Probably a new DOI should be created for the current 1.0.1 version and have both links updated before paper is published.
from joss-reviews.
Ok @arfon and @xuanxu I have made an updated release with a new zenodo id 10.5281/zenodo.51798 but this is in the whedon part of the submission not the paper.md so I can't see a way for me to edit it.
Updated the how to cite to reflect JOSS, and the Zenodo badge updates automatically (links to current version not a fixed DOI).
from joss-reviews.
All checked!
from joss-reviews.
Thanks @xuanxu
@arfon PDF will need regenerating. Look forward to being official! Where do I sign up as a reviewer and evangelist?
from joss-reviews.
Thanks @xuanxu for the review.
@arfon PDF will need regenerating.
@genomematt - your DOI will be http://dx.doi.org/10.21105/joss.00018 but the Crossref system seems to be a little backed up right now (this paper is pretty far back in the queue).
Where do I sign up as a reviewer and evangelist?
I'd be happy to add you to the @openjournals/joss-reviewers
team if that sounds good?
from joss-reviews.
@arfon Only the second affiliation comes up on the PDF. I will raise a bug.
Yes, add me to @openjournals/joss-reviewers
from joss-reviews.
Yes, add me to
@openjournals/joss-reviewers
Excellent, thanks! You should have received an invite to join the team.
from joss-reviews.
@genomematt - the DOI is now live: http://dx.doi.org/10.21105/joss.00018
from joss-reviews.
from joss-reviews.
Related Issues (20)
- [PRE REVIEW]: prospect: an R package to link leaf optical properties with their chemical and structural properties with the leaf model PROSPECT HOT 9
- [PRE REVIEW]: automata: A Python package for simulating and manipulating automata HOT 16
- [REVIEW]: matscipy: materials science at the atomic scale with Python HOT 5
- [REVIEW]: OceanBioME.jl: A flexible environment for modelling the coupled interactions between ocean biogeochemistry and physics HOT 18
- [REVIEW]: Pycapacity: a real-time task-space capacity calculation package for robotics and biomechanics HOT 8
- [REVIEW]: UQTestFuns: A Python3 library of uncertainty quantification (UQ) test functions HOT 7
- [PRE REVIEW]: GPU Code Generation of Cardiac Electrophysiology Simulation with MLIR HOT 12
- [REVIEW]: PurpleCaffeine: tracking of quantum programs and experiments HOT 11
- [PRE REVIEW]: BlackBIRDS: Black-Box Inference foR Differentiable Simulators HOT 10
- [PRE REVIEW]: Using the tidyverse with terra objects: the tidyterra package HOT 20
- [PRE REVIEW]: measr: Bayesian psychometric measurement using Stan HOT 8
- [PRE REVIEW]: Iterative-stats: A Python package for basic iterative statistics HOT 10
- [PRE REVIEW]: Bibat: batteries-included Bayesian analysis template HOT 11
- [PRE REVIEW]: FluxNLPModels.jl and KnetNLPModels.jl: Connecting Deep Learning Models with Optimization Solvers HOT 13
- [PRE REVIEW]: epiworldR: Fast Agent-Based Epi Models HOT 11
- [PRE REVIEW]: ZOSPy: optical ray tracing in Python through OpticStudio HOT 14
- [PRE REVIEW]: OpenSkill: A faster asymmetric multi-team, multiplayer rating system HOT 9
- [PRE REVIEW]: StableRLS: Stable Reinforcement Learning for Simulink HOT 7
- [PRE REVIEW]: TensorInference: A Julia package for tensor-based probabilistic inference HOT 41
- [PRE REVIEW]: concstats: Market Structure, Concentration and Inequality Measures, with R HOT 7
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from joss-reviews.