-
-
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]: spaghetti: spatial network analysis in PySAL #2826
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @martibosch it looks like you're currently assigned to review this paper 🎉. Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post. ⭐ 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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
Still working on getting the second reviewer. @martibosch -- you're free to go ahead with your review. |
@whedon generate pdf from branch joss |
|
|
Hello all, I am sorry for the delay in reviewing this submission. I am at the end of my thesis and this ended up buried in a pile of "TODOs". If this works for everybody, will review the submission by the end of this week. Or should I wait for the amendments that @jGaboardi is planning to include? How do we proceed regarding that, @usethedata? Best, |
The release is a scheduled one to coincide with the biannual PySAL federation release. It will not include any change to the actual code-base or the text of the JOSS paper, just admin/maintenance adjustments for testing, docs, etc. I will get that out today, which will be update: |
This would be all from my side. Looking forward to hear back from the authors. Let me clear out again that my intention is not to question the criteria of the authors (since they probably know better than me why they have made such choices), but rather ensure that the raised issues have been considered. Best, |
Thanks for the thorough review, @martibosch. We will go through the issue you raised and respond ASAP. |
@martibosch -- Thanks. |
@jGaboardi – At this point could you make a new release of this software that includes the changes that have resulted from this review. Then, please make an archive of the software in Zenodo/figshare/other service and update this thread with the DOI of the archive? For the Zenodo/figshare archive, please make sure that:
I can then move forward with accepting the submission. |
@arfon I have cut a fresh release and Zenodo archive with the following info. Let me know if I did this correctly.
|
@whedon set 10.5281/zenodo.4898079 as archive |
OK. 10.5281/zenodo.4898079 is the archive. |
@whedon set v1.5.8 as version |
OK. v1.5.8 is the version. |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2357 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2357, then you can now move forward with accepting the submission by compiling again with the flag
|
@jGaboardi – please merge this PR which makes a small number of formatting fixes and removes a duplicate sentence pysal/spaghetti#626. |
@arfon pysal/spaghetti#626 is now merged. |
@whedon generate pdf |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2358 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2358, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true |
|
🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦 |
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨 Here's what you must now do:
Any issues? Notify your editorial technical team... |
@martibosch, @usethedata – many thanks for your reviews here and to @usethedata for editing too! JOSS relies upon the volunteer efforts of people like you and we simply wouldn't be able to do this without you ✨ On a personal note, thanks to @usethedata who is stepping down from the JOSS editorial team. Thanks Bruce for all of your help making JOSS the success it is 💖 @jGaboardi – your paper is now accepted and published in JOSS ⚡🚀💥 |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
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: @jGaboardi (James Gaboardi)
Repository: https://github.com/pysal/spaghetti
Version: v1.5.8
Editor: @usethedata
Reviewers: @martibosch, @usethedata
Archive: 10.5281/zenodo.4898079
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
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) by leaving comments 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
@martibosch, 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.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @usethedata know.
✨ Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest ✨
Review checklist for @martibosch
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @usethedata
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: