Comments (21)
@drvinceknight, @sdross0 — Thank you both for reviewing this submission to JOSE. Since you both have ticked all items in your checklist, and made constructive comments that were addressed, all I need now is a confirmation on a comment below that you are ready to recommend acceptance.
This will be the first paper in JOSE and we're excited and grateful to you for being a part of this plunge into a new model of scholarly publication!
from jose-reviews.
@gboeing -- your paper is now published in JOSE
You can now add the green happy JOSE button on your repository.
@drvinceknight, @sdross0 — Thank you again for your review
from jose-reviews.
@sdross0 thank you for these thoughtful recommendations. I have made each of these changes you suggested:
- clarified the software's focus on 1-D maps in the documentation in gboeing/pynamical@22a3e7f and readme in gboeing/pynamical@2a1cad1
- identified where this software could be most useful, accordingly, in the beginning of an intro course or demonstrating patterns to students in less technical fields, in gboeing/pynamical@471f11e
- highlighted example of how to define/input your own model in gboeing/pynamical@2a1cad1 (example demonstrates defining the mandelbrot map then inputting it for simulation/visualization)
from jose-reviews.
from jose-reviews.
Proof looks good, I will complete the review before the weekend.
from jose-reviews.
now is a confirmation on a comment below that you are ready to recommend acceptance.
I confirm
This will be the first paper in JOSE and we're excited and grateful to you for being a part of this plunge into a new model of scholarly publication!
Awesome:
from jose-reviews.
I recommend acceptance.
from jose-reviews.
@gboeing The next step is for you to make an archive on a service like Zenodo, get a DOI, and post the DOI here. (You may want to tag a release on your GitHub repo right before archiving.)
from jose-reviews.
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks. @drvinceknight it looks like you're currently assigned as the reviewer for this paper
If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/jose-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews
To fix this do the following two things:
- Set yourself as 'Not watching' https://github.com/openjournals/jose-reviews:
- You may also like to change your default settings for this watching repositories in your GitHub profile here: https://github.com/settings/notifications
For a list of things I can do to help you, just type:
@whedon commands
from jose-reviews.
Attempting PDF compilation. Reticulating splines etc...
from jose-reviews.
I'm (really) impressed with the software and the paper is well written.
A couple of minor things that I have not checked off on my list and would like to run past @gboeing:
Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)?
The documentation on github is great: I ran everything and it worked exactly as expected.
I noted that the documentation rendered by sphinx isn't complete. The autodoc of the api doesn't seem to have picked up the various methods/modules: https://pynamical.readthedocs.io/en/latest/pynamical.html#submodules
Tests: Are there automated tests or manual steps described so that the function of the software can be verified?
The test suite is currently just a test of run to completion of the various functions and not of expected output. I don't think this is something that should hinder publication at all but perhaps @gboeing you would want to think about adding some tests of expected output:
For example:
def test_phase_diagram_3d():
pops = simulate(model=cubic_map, num_gens=200, rate_min=3.5, num_rates=30, num_discard=100)
phase_diagram_3d(pops, xmin=-1, xmax=1, ymin=-1, ymax=1, zmin=-1, zmax=1, alpha=0.2, color='viridis', azim=330,
legend=True, save=False, folder=_img_folder, filename='')
phase_diagram_3d(pops, xmin=-1, xmax=1, ymin=-1, ymax=1, zmin=-1, zmax=1, alpha=0.2, color='inferno', azim=330,
legend=True, remove_ticks=False, save=False, folder=_img_folder, filename='')
we could at least assert the type of pops
(that it returns a dataframe) and possibly also the shape of the dataframe? Similarly for phase_diagram_3d
(we can assert that we get a tuple, and the type of each element).
Community guidelines: 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
I didn't see these in the repo but perhaps I missed them?
from jose-reviews.
@drvinceknight thank you for these thoughtful recommendations. I have made each of these changes you suggested:
- sphinx autodoc now renders all the docstrings properly: https://pynamical.readthedocs.io/en/latest/pynamical.html#submodules
- new tests accordingly added in gboeing/pynamical@861f9ce
- community contribution, issues, and support guidelines added in gboeing/pynamical@689cea0
from jose-reviews.
Thanks for those @gboeing: it all looks great to me know and I gladly recommend this for publication. A very nice library, I might make use of it myself at some point (will let you know if/when I do!).
from jose-reviews.
@drvinceknight thanks!
from jose-reviews.
Thanks for your review, @drvinceknight
@sdross0 — Since we have a thorough review of the software side by @drvinceknight, I encourage you to focus your review on the educational value of the package, and especially go over the demos/tutorial and see if you have any recommendations there. As for installation, etc., you could have a student in your group install it, to get a double check and tick off those items on the review checklist. Thank you!
from jose-reviews.
Pros:
The software installs as advertised through both methods, and is able to successfully run all examples presented. Quick set-up and first examples worked out of the box. It runs very fast, which impresses me a lot. Very nice.
Cons:
From the docs, it's not clearly explained what class of problems this is designed to work for. Nonlinear dynamical systems could be any number of things (n-dimensional mappings, systems of ordinary differential equations, time-delay equations, etc.)
It seems to be designed primarily for 1D maps, that is, 1-dimensional discrete dynamical systems of the form
x_k+1 = f(x_k)
which isn't explicitly described in the documentation. I think a description of the kinds of models that the software has in mind will make it more successful. It would also help to know how to input your own models into the system.
Related to that, I worry the software might not generalize well to the other types of dynamical systems that would appear in an introductory course on dynamical systems (say, following Strogatz book), mostly because of the restriction to 1D maps. Most such courses do include 1D maps at the beginning of the course, and perhaps that is where Pynamical could be most useful.
from jose-reviews.
@gboeing See how you can address the comments above in the documentation, examples and paper, and let us know. You may want to add a note about the current limitations of Pynamical, whether and how it could be expanded beyond those limitations, and how it might be used in a standard course in dynamical systems.
from jose-reviews.
@labarba 10.5281/zenodo.1294299
from jose-reviews.
@whedon set 10.5281/zenodo.1294299 as archive
from jose-reviews.
OK. 10.5281/zenodo.1294299 is the archive.
from jose-reviews.
If you would like to include a link to your paper from your README use the following code snippet:
[![DOI](https://jose.theoj.org/papers/10.21105/jose.00015/status.svg)](https://doi.org/10.21105/jose.00015)
This is how it will look in your documentation:
We need your help!
Journal of Open Source Education is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
- Volunteering to review for us sometime in the future. You can add your name to the reviewer list here: http://jose.theoj.org/reviewer-signup.html
- Making a small donation to support our running costs here: https://www.flipcause.com/secure/cause_pdetails/Mjk3ODA=
from jose-reviews.
Related Issues (20)
- [REVIEW]: Course Materials for an Introduction to Data-Driven Chemistry HOT 117
- [REVIEW]: The Argo Online School: An e-learning tool to get started with Argo HOT 35
- @whedon, this comment explains my review of the JOSE submission #173. I was invited to review this paper by @allisonhorst . HOT 1
- [PRE REVIEW]: R for Data Analysis: An open-source resource for teaching and learning analytics with R HOT 49
- [REVIEW]: Special relativity in R: the lorentz package HOT 21
- [REVIEW]: FANGS - Fire Applications with Next-Generation Satellites HOT 20
- [REVIEW]: Ecological Forecasting and Dynamics: A graduate course on the fundamentals of time series and forecasting in ecology HOT 87
- [PRE REVIEW]: Manim Slides: A Python package for presenting Manim content anywhere HOT 16
- [REVIEW]: Module on dust aerosol detection, monitoring and forecasting HOT 17
- [PRE REVIEW]: Planet_LB: Lattice-Boltzmann solutions for planetary geodynamics problems HOT 25
- [REVIEW]: R for Data Analysis: An open-source resource for teaching and learning analytics with R HOT 47
- [PRE REVIEW]: ChooChoo the Checklist tool HOT 33
- [PRE REVIEW]: A Data Carpentry- Style Metagenomics Workshop HOT 22
- [REVIEW]: Planet_LB: Lattice-Boltzmann solutions for planetary geodynamics problems HOT 95
- [REVIEW]: Manim Slides: A Python package for presenting Manim content anywhere HOT 71
- Tests: Are there automated tests or manual steps described so that the function of the software can be verified? HOT 1
- [REVIEW]: A Data Carpentry- Style Metagenomics Workshop HOT 73
- [PRE REVIEW]: languagemodels: A Python Package for Exploring Modern Natural Language Processing HOT 15
- [PRE REVIEW]: Check your outliers! An introduction to identifying statistical outliers in R with *easystats* HOT 58
- [PRE REVIEW]: An R Companion for Introduction to Data Mining HOT 52
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 jose-reviews.