-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
[REVIEW]: ChebTools: C++11 (and Python) tools for working with Chebyshev expansions #569
Comments
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 🎉. ⭐ Important ⭐ If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-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:
For a list of things I can do to help you, just type:
|
|
|
OK, @drvinceknight: here's where the action happens. If you've reviewed for JOSS before, you know the drill. If not, I'm here to answer your questions! |
@ianhbell -- Your paper is not compiling. Can you check on that? |
I don't think that is related to my paper. I don't do anything too crazy, perhaps it is related to my Python code block? I don't have any way of testing compilation on my side. I asked several times about that but was never able to get a working build to the journal specs on my machine. I think the whedon server needs to have the LaTeX package added. |
@arfon : Can you help here? The paper is not compiling. |
@whedon generate pdf |
|
|
@labarba @ianhbell - looks like our @whedon LaTeX environment doesn't include For now will this copy of the paper work that I compiled locally? 10.21105.joss.00569.pdf |
That local copy looks beautiful! How about making a docker container with whedon inside? That might be the cleanest packaging. I can try to help with that if you are interested? |
I haven't reviewed for JOSS before but everything looks pretty self explanatory (I'm just sat here thinking how great the whole system is!). I'm aiming to complete this over next couple of days. |
Installation
My suggestion would be to modify the install instructions to make the DocumentationAs far as I can see there's no simple example usage in the There is also no "A statement of need" in the documentation ( Community
This seems to be missing. Release version@labarba I'm not entirely sure I follow this:
As far as I can tell this software is "only" released on github. Is this requiring/expecting the software itself to be archived on a service like Zenodo? I haven't been able to test any of the Functionality aspects, will do that once I've understood what I've missed with the installation. @ianhbell I hope this is helpful and please let me know if I've misunderstood anything. |
@drvinceknight : The software will need to be archived on Zenodo or similar, and get a DOI. But we've relaxed the requirement to have that before acceptance, because when the review elicits changes, this leads to a new release. We ask that authors deposit the accepted version. |
@drvinceknight - the build problem you had was almost certainly because you didn't do a recursive git clone - I've been bitten by that innumerable times. I updated the docs to try to address that issue and I tried to address your other concerns as well. What do you think now: https://github.com/usnistgov/ChebTools ? Ready for any other review you care to offer. |
And yes, I have minted a doi on zenodo already, but I'll remake the release when the review is complete. |
@ianhbell the docs look great! Thanks for those changes. I'm still having issues installing on my system though:
I went for the |
Sorry I forgot to answer about that import error. That's a bug in anaconda... : ContinuumIO/anaconda-issues#483 The short answer is : delete the libstdc++ that it references, and it will then hook your system GLIBC and it should all work fine, though you might want to backup your libstdc++ before you do that :) |
Running |
Sure, I can do that
|
Added to the README. |
I take it your recommendation is to accept the submission, at this point, @drvinceknight ? |
Yes, my recommendation is to accept the submission. |
All done! DOI: 10.5281/zenodo.1172036 |
@whedon set 10.5281/zenodo.1172036 as archive |
OK. 10.5281/zenodo.1172036 is the archive. |
@drvinceknight - many thanks for your review here and to @labarba for editing this submission ✨ @ianhbell - your paper is now accepted into JOSS and your DOI is https://doi.org/10.21105/joss.00569 ⚡️ 🚀 💥 |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippet:
This is how it will look in your documentation: We need your help! Journal of Open Source Software 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:
|
Submitting author: @ianhbell (Ian Bell)
Repository: https://github.com/usnistgov/ChebTools
Version: v1.0.0
Editor: @labarba
Reviewer: @drvinceknight
Archive: 10.5281/zenodo.1172036
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@drvinceknight, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @labarba know.
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?The text was updated successfully, but these errors were encountered: