-
-
Notifications
You must be signed in to change notification settings - Fork 36
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]: Spectral Connectivity: a python package for computing multitaper spectral estimates and frequency-domain brain connectivity measures on the CPU and GPU #4840
Comments
Hello humans, I'm @editorialbot, a robot that can help you with some common editorial tasks. 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:
|
|
Wordcount for |
|
Hi @AJQuinn, @sappelhoff, and @EtienneCmb 👋 Thanks again for agreeing to review this submission ! The review will take place in this issue, and you can generate your individual reviewer checklists by asking editorialbot directly with In working through the checklist, you're likely to have specific feedback on Spectral Connectivity. Whenever possible, please open relevant issues on the linked software repository (and cross-link them with this issue) rather than discussing them here. This helps to make sure that feedback is translated into actionable items to improve the software ! If you aren't sure how to get started, please see the Reviewing for JOSS guide -- and, of course, feel free to ping me with any questions ! |
Review checklist for @sappelhoffConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@editorialbot generate my checklist |
Hm, you might need to just create a new comment @AJQuinn -- I'm not sure that it will respond to edited comments ! Sorry for the confusion. |
Review checklist for @AJQuinnConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@emdupre I have completed my review. All issues that I raised have been addressed to my full satisfaction. I believe that For details, please see my checklist and all my linked issues and PRs in this review thread. |
@editorialbot set 10.5281/zenodo.7416614 as archive |
Done! Archive is now 10.5281/zenodo.7416614 |
Thank you, @edeno ! I noticed that the metadata on the archive has several small discrepancies from the paper metadata. In particular, could you please update the title to :
and double-check that the author information lists names and surnames correctly ? From my rendering, these appear inverted in the final citation (i.e., excerpting from the bib file):
You can update these without creating a new archive by editing the existing archive's metadata as described here. |
Ah yes. Silly mistake by me. Should be fixed now. |
Thank you ! I see that the author name changes have updated but not the title of the archive. Could you please confirm that ? |
Sorry, I misunderstood what the title referred to. It should be fixed now? |
Thank you ; this now looks right on my end ! I'm now happy to recommend Spectral Connectivity to the EIC team for publication, and I just want to add my congratulations to you on such an impressive effort ! 🎉 |
@editorialbot recommend-accept |
|
|
👋 @openjournals/bcm-eics, this paper is ready to be accepted and published. Check final proof 👉📄 Download article If the paper PDF and the deposit XML files look good in openjournals/joss-papers#3787, then you can now move forward with accepting the submission by compiling again with the command |
I found a couple of small typos that I fixed. We are happy how it looks otherwise. |
@edeno I am the AEiC for this track and here to help process this work for acceptance in JOSS. I have reviewed the archive and the paper and all seems in order. I only have one minor editorial point:
Please let me know when you've worked on the above. Thanks. |
I added the affiliation country to the archive and the paper.md. Please let me know if there's another place it should be added. |
@editorialbot generate pdf |
@edeno all looks good now so we'll proceed with acceptance. Thanks |
@editorialbot accept |
|
🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦 |
🐘🐘🐘 👉 Toot 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... |
@edeno congratulations on this JOSS publication! 🎉 Thanks @emdupre for editing! And also thank you to the reviewers @AJQuinn, @sappelhoff, and @EtienneCmb!!!!! |
🎉🎉🎉 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! The 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: @edeno ()
Repository: https://github.com/Eden-Kramer-Lab/spectral_connectivity
Branch with paper.md (empty if default branch):
Version: v1.1.0
Editor: @emdupre
Reviewers: @AJQuinn, @sappelhoff, @EtienneCmb
Archive: 10.5281/zenodo.7416614
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
@AJQuinn & @sappelhoff & @EtienneCmb, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review.
First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @emdupre 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 ✨
Checklists
📝 Checklist for @AJQuinn
📝 Checklist for @sappelhoff
📝 Checklist for @EtienneCmb
The text was updated successfully, but these errors were encountered: