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]: udpPacketManager: An International LOFAR Station Data (Pre-)Processor #5517
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 |
|
Review checklist for @pritchardnConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hello all I plan on finishing my review by the end of June and will add comments as I see them. |
@shmookey when you get a chance, please reply to this comment with:
This will create your reviewer checklist that you can then begin going through. Please let me know if you have any questions! |
A few minor comments on the Software Paper (mainly typos) which is otherwise excellent:
|
I've tried to install the library today, but a problem installing psrdada is preventing me. |
Hey @pritchardn, Thanks! I've made changes to the paper and the general documentation comments you made in the docsPass branch (PR David-McKenna/udpPacketManager#13 ). As for the PSRDADA issue, are sourceforge's certs out of date again? That's been a chronic issue over the past few years. I'll see if there's another way for me to download their source in my CMake script. Cheers, |
* General documentation pass (following #12) MkDocs Config (will be hosted at upm-docs.pulsar-observer) Additional note on intended audience in main README Fix-up some typos, missing JOSS information Fix a mistake in README_CLI.md * Typo fix; further Docker/Singularity information * Incorporate @pritchardn's feedback from openjournals/joss-reviews#5517 * Change outdated help message for "-c" in CLIs
@shmookey when you get a chance, please begin your review of this package. You can get your reviewer checklist by responding to this thread with:
If you feel that you are no longer able to review, please let me know, and I can work on finding another reviewer. Thanks! |
@plaplant – I think it might be time to find a different reviewer here? |
@arfon yes, I agree. I've actually been working behind the scenes to reach out to other potential reviewers, but have been coming up empty. I will reach out to additional folks and see if I can get someone to volunteer. Thanks for keeping up with this! |
Given that this submission has languished a bit, I'm going to go ahead and step in as a reviewer here to make sure the submission keeps moving forward. To keep things conflict-of-interest-free, @dfm has graciously agreed to take over as editor. I will review this as soon as I can, and hopefully we can get this fully reviewed soon. @David-McKenna thanks so much for your patience thus far! Please let me (or @dfm) know if you have any questions. |
@editorialbot assign @plaplant as reviewer |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
@editorialbot add @plaplant as reviewer |
@plaplant added to the reviewers list! |
|
@pritchardn, @plaplant — Thanks for your thorough and constructive reviews!! @David-McKenna — I've opened a small PR with some minor edits to the manuscript, please take a look and merge or let me know what you think. Once you've done that:
|
@David-McKenna — Checking in here. We're so close to publication, could up update me on the status of these steps that I asked for above? |
Hey @dfm, Apologies, I apparently forgot to swap my GitHub account to my new work email so I never saw the prior comments. I'll get this handled by the end of today. |
Tagged and released 0.9.2 with your proposed changes, the Zenodo DOI is 10.5281/zenodo.11019139. Cheers, |
@David-McKenna — Thanks! Can you update the metadata for the Zenodo deposit (there should be an "edit" button on the top right corner of that page) so that the title and author list match the paper? |
Whoops, should be fixed up now. |
@editorialbot set 10.5281/zenodo.11019139 as archive |
Done! archive is now 10.5281/zenodo.11019139 |
@editorialbot set 0.9.2 as version |
Done! version is now 0.9.2 |
@editorialbot generate pdf |
@editorialbot recommend-accept |
|
👋 @openjournals/aass-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#5259, then you can now move forward with accepting the submission by compiling again with the command |
|
@editorialbot accept |
|
Ensure proper citation by uploading a plain text CITATION.cff file to the default branch of your repository. If using GitHub, a Cite this repository menu will appear in the About section, containing both APA and BibTeX formats. When exported to Zotero using a browser plugin, Zotero will automatically create an entry using the information contained in the .cff file. You can copy the contents for your CITATION.cff file here: CITATION.cff
If the repository is not hosted on GitHub, a .cff file can still be uploaded to set your preferred citation. Users will be able to manually copy and paste the citation. |
🐘🐘🐘 👉 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... |
@pritchardn, @plaplant — Many thanks for your reviews here! And also thanks to @plaplant for getting the review started as editor. JOSS relies upon the volunteer effort of people like you and we simply wouldn't be able to do this without you!! @David-McKenna — 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! 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: @David-McKenna (David McKenna)
Repository: https://github.com/David-McKenna/udpPacketManager/
Branch with paper.md (empty if default branch):
Version: 0.9.2
Editor: @dfm
Reviewers: @pritchardn, @plaplant
Archive: 10.5281/zenodo.11019139
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
@shmookey & @pritchardn, 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 @plaplant 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 @pritchardn
📝 Checklist for @plaplant
The text was updated successfully, but these errors were encountered: