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]: Stripy: A Python module for (constrained) triangulation in Cartesian coordinates and on a sphere #1410
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @santisoler 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:
|
|
PDF failed to compile for issue #1410 with the following error: /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in |
@whedon generate pdf from branch paper |
|
PDF failed to compile for issue #1410 with the following error: sh: 1: cd: can't cd to tmp/1410 |
@VivianePons seems like |
Hi @lmoresi and @brmather. I was assigned as reviewer for your submitted paper. I'm still thinking my review strategy, but I'll probably will open issues on your repo regarding each section of the checklist (e.g. General Checks, Functionality, Documentation, Software Paper). I'm sure this will be a nice experience for all of us. Be patient, I'll contact you when I have some news. |
Thanks - we’ll work with the way you prefer to review. issues sounds like the obvious strategy. We have a “paper” branch of the code to make sure that any developments we make during the review process do not cause problems for you. Any suggestions you make st this stage will be merged back when the process is done.
Prof Louis Moresi
louis.moresi@unimelb.edu.au<mailto:louis.moresi@unimelb.edu.au>
(w) +61 3 8344 1217
(m) +61 4 0333 1413
(us) +1 505 349 4425
www.moresi.info<http://www.moresi.info/>
www.facebook.com/underworldcode<http://www.facebook.com/underworldcode>
@LouisMoresi<https://twitter.com/LouisMoresi>
On 24 Apr 2019, 07:20 -0700, Santiago Soler <notifications@github.com>, wrote:
Hi @lmoresi<https://github.com/lmoresi> and @brmather<https://github.com/brmather>. I was assigned as reviewer for your submitted paper.
Let me read through your PDF, explore your stripy repo and experiment with your software.
I'll try to meet the two weeks deadline, but in case I get delayed I'll let you know so you don't get worried about the status of the review.
I'm still thinking my review strategy, but I'll probably will open issues on your repo regarding each section of the checklist (e.g. General Checks, Functionality, Documentation, Software Paper).
Besides that, I'm open to help you solve the issues that may appear. My goal is to contribute to get this published! So don't hesitate asking for help or guidance, I'm willing to help with everything that's on my reach.
I'm sure this will be a nice experience for all of us. Be patient, I'll contact you when I have some news.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#1410 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ADABPI5V7F7RR6NJBZEVFJLPSBUAVANCNFSM4HID64OA>.
|
@whedon generate pdf from branch paper |
|
Thanks for pointing this out. I've started reviewing some of the more basic aspects of the publication. |
Hi @VivianePons. I have a small question regarding how versioning works on the review process. |
Yes, don't worry about the version. When the paper is approved, we update the version before the final publication |
I have a question for @arfon : I have contacted Robert Renka to be a second reviewer on this paper (as suggested by the author). He would be willing to do it but he is also the author of the original Fortran code on which the software is based. He is wondering if this is a case of conflict of interest. I believe it's ok. I actually think that his opinion on the paper would be very valuable. Besides, we have a second independent reviewer so the publication is not based on his opinion alone. What do you think? |
👍 this sounds OK to me. Thanks for checking. |
Thanks for answering this @VivianePons! |
OK, @rrenka is now a reviewer |
@whedon generate pdf from branch master |
|
|
Check final proof 👉 openjournals/joss-papers#755 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#755, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon generate pdf |
|
@whedon accept |
|
|
Check final proof 👉 openjournals/joss-papers#756 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#756, 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... |
@santisoler, @rrenka - many thanks for your reviews and to @VivianePons for editing this submission ✨ @lmoresi - your paper is now accepted into 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:
|
The Software repository link on the final article is 404. It should probably point to the repo, not a subdirectory. |
I have added a ‘paper’ tag to the repo that snapshots the current / published state so that the link no longer fails.
However, this is a snapshot of the published release and I am not sure that this is what is intended by that link.
Prof Louis Moresi
louis.moresi@unimelb.edu.au<mailto:louis.moresi@unimelb.edu.au>
(w) +61 3 8344 1217
(m) +61 4 0333 1413
(us) +1 505 349 4425
www.moresi.info<http://www.moresi.info/>
www.facebook.com/underworldcode<http://www.facebook.com/underworldcode>
@LouisMoresi<https://twitter.com/LouisMoresi>
On 14 Jun 2019, 5:27 AM +0100, Elliott Sales de Andrade <notifications@github.com>, wrote:
The Software repository link on the final article is 404. It should probably point to the repo, not a subdirectory.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#1410?email_source=notifications&email_token=ADABPI7KPCXRBCY5WU3IKXTP2MM27A5CNFSM4HID64OKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXVVS2I#issuecomment-501963113>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ADABPI2JNT4NTI6PLGAIFPDP2MM27ANCNFSM4HID64OA>.
|
This is fixed now. Thanks for catching this. |
Submitting author: @lmoresi (Louis Moresi)
Repository: https://github.com/underworldcode/stripy
Version: 1.0.1
Editor: @VivianePons
Reviewers: @santisoler, @rrenka
Archive: 10.5281/zenodo.3243511
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
@santisoler & @rrenka, 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 @VivianePons know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @santisoler
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?Review checklist for @rrenka
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: