-
-
Notifications
You must be signed in to change notification settings - Fork 39
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]: PFEIFER: Preprocessing Framework of Electrograms Intermittently Fiducialized from Experimental Recordings #472
Comments
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks. @CristianaCorsi 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 as 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:
|
|
1 similar comment
|
|
1 similar comment
|
@CristianaCorsi this is where the review takes place. The review is guided by the tick boxes at the top of this issue which you are able to tick. See our website and reviewer guidelines for more information. If you have comments or issues you want to report for the authors you can leave comments here. You can also post issues on the project repo and link to them here. Thanks you. Let me know if I can help with anything. |
@CristianaCorsi could you provide an indication as to when you think you can complete this review? Thanks |
@bzenger The initial paper generation did not complete. I am unsure about the error but it looks like it cannot find your paper file. Could you please rename the paper folder and paper file to |
@Kevin-Mattheus-Moerman This should be updated now. Could we re-run the generation function? |
@whedon generate pdf |
|
|
@bzenger that seems to have worked. Can you check if the affiliations are complete/correct in your article? |
@Kevin-Mattheus-Moerman I just updated the affiliations for clarifications. If you could have whedon run again that would be great! Thank you so much! |
@whedon generate pdf |
|
|
@whedon generate pdf |
|
|
@whedon generate pdf |
|
|
@bzenger - sorry for the unhelpful errors from @whedon. I've made some fixes to your paper YAML header that were obvious (to me 😄) problems: SCIInstitute/PFEIFER#24 |
|
|
@arfon. Perhaps we can fix this by always starting the DOI link on a new line. |
Perhaps. Not sure how we would do that though? |
I've added some statements over at openjournals/whedon#14. |
@CristianaCorsi how are you getting on? Let me know if I can help with anything. |
Hi. I downloaded all the material. I hope I will be able to provide the review in ten days. If I need help I will let you know. Thank you! |
@CristianaCorsi thanks for the update! Our review guidelines are here and you can find the tick-boxes at the top of this issue. Thanks for your help reviewing! |
@bzenger @CristianaCorsi Happy New Year! 🎉 |
Hi Kevin.
I completed the review process.
I checked all the flags in the checklist and added some comments to the
Documentation section.
In my opinion the paper should be accepted.
Is the review process finalized or I need to upload some additional
files? Do I need to apply some different label to this issue?
Thank you!
Cristiana
Il 08/01/2018 17.22, Kevin Mattheus Moerman ha scritto:
…
@bzenger <https://github.com/bzenger> @CristianaCorsi
<https://github.com/cristianacorsi> Happy New Year! 🎉
@CristianaCorsi <https://github.com/cristianacorsi> how are you
getting on? Are you able to finalize the review process soon? Let me
know if I can help with anything.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#472 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AgVBg6ajVu7xOxAPXiCAJ4R1py0nrtNzks5tIkDSgaJpZM4Qr2eU>.
--
Cristiana Corsi, PhD, Associate Professor
Department of Electric, Electronic and Information Engineering "Guglielmo Marconi" - DEI
University of Bologna
Via Risorgimento 2 - 40136 Bologna - Italy
Via Venezia 52 - 47521 Cesena - Italy
tel. +39 051 2093009 - +39 0547 339207
|
@ChristianaCorsi 🎉 excellent thanks for completing the review process. We will handle things from here on. Can you and @bzenger confirm the comments to the documentation have been implemented? Can you link to an issue (if an issue was used)? |
Hi Kevin,
We just updated our documentation with some small changes. These were done over an email exchange not through any GitHub issue.
Dr. Corsi will email you with the confirmation.
Thanks!
Brian
… On Jan 22, 2018, at 10:11 AM, Kevin Mattheus Moerman ***@***.***> wrote:
@christianacorsi 🎉 excellent thanks for completing the review process. We will handle things from here on. Can you and @bzenger <https://github.com/bzenger> confirm the comments to the documentation have been implemented? Can you link to an issue (if an issue was used)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#472 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AdF4h-NS-c7f7SUyAa-Bu-AJ8NxNsPayks5tNMFHgaJpZM4Qr2eU>.
|
All the comments to the documentation have been implemented.
Brian, could you please update the DOC folder in the submission web site
with this new file (now available on the server
http://sci.utah.edu/devbuilds/pfeifer_docs/PfeiferDocumentation.pdf)?
Thank you!
Cristiana
Il 22/01/2018 18.11, Kevin Mattheus Moerman ha scritto:
…
@christianacorsi 🎉 excellent thanks for completing the review
process. We will handle things from here on. Can you and @bzenger
<https://github.com/bzenger> confirm the comments to the documentation
have been implemented? Can you link to an issue (if an issue was used)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#472 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AgVBg6yr94HdduiWxJUlsPFtt7DvZTB5ks5tNMFHgaJpZM4Qr2eU>.
--
Cristiana Corsi, PhD, Associate Professor
Department of Electric, Electronic and Information Engineering "Guglielmo Marconi" - DEI
University of Bologna
Via Risorgimento 2 - 40136 Bologna - Italy
Via Venezia 52 - 47521 Cesena - Italy
tel. +39 051 2093009 - +39 0547 339207
|
@bzenger can you respond to @CristianaCorsi 's comments? |
@Kevin-Mattheus-Moerman @CristianaCorsi the doc is updated in the github repo. Thanks! |
@Kevin-Mattheus-Moerman let us know if there is anything else you need us to do! |
Okay it looks like we are all set here! 🚀 🎉 |
@Kevin-Mattheus-Moerman is this all I need for that process? |
Yes all set. @arfon will take it from here. |
@whedon set 10.5281/zenodo.1158651 as archive |
OK. 10.5281/zenodo.1158651 is the archive. |
@CristianaCorsi - many thanks for your review here and to @Kevin-Mattheus-Moerman for editing this one ✨ @bzenger - your paper is now accepted into JOSS and your DOI is https://doi.org/10.21105/joss.00472 ⚡️ 🚀 💥 |
🎉🎉🎉 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 volunteering to review for us sometime in the future. You can add your name to the reviewer list here: http://joss.theoj.org/reviewer-signup.html |
Submitting author: @bzenger (Brian Zenger)
Repository: https://github.com/SCIInstitute/PFEIFER
Version: v1.0.0
Editor: @Kevin-Mattheus-Moerman
Reviewer: @CristianaCorsi
Archive: 10.5281/zenodo.1158651
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
@CristianaCorsi, 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: http://joss.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @Kevin-Mattheus-Moerman know.
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
I suggest to use in the Documentation, figures that describe exactly the windows that open when you run PFEIFER. For example, in Chapter 6, figure 6.1 shows a window slightly different from the DATA_ORGANIZATION window that opens when you run PFEIFER. In the window each section has a specific name and the BEAT PROPAGATION section includes six fields, not four as reported in the figure (Window Length is called Window Width; the fields #Beats before updating and # Beats for updating are missing as well as the flags “Include RMS for Autofiducializing”, “Local Fiducials”, “Update Templates”; and the button !Default Settings!). The design of the window is also different from the one reported in figure 6.1.
I also suggest to include in the Documentation the link to the tutorials available at: http://www.sci.utah.edu/software/pfeifer.html
Software paper
paper.md
file include a list of authors with their affiliations?The text was updated successfully, but these errors were encountered: