-
-
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]: EUKulele: Taxonomic annotation of the unsung eukaryotic microbes #2817
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @johanneswerner, @jcmcnch 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:
|
|
👋 @jcmcnch, please update us on how your review is going. |
👋 @johanneswerner, please update us on how your review is going. |
Very interesting software package for the analysis of eukaroytes in metagenomes and metatranscriptomes. I like the focus of this tool and the well-written article and documentation, especially the very comprehensive documentation including all explanations and citations. I have a few comments that might still be addressed.
|
Great - thanks @johanneswerner! Can you please respond to these comments when you get the chance @akrinos. @jcmcnch - can you let us know how your are getting on please? |
@johanneswerner Thank you so much for the very helpful review! I will respond to what I have responses for thus far and update as additional comments are addressed.
Other Questions
|
@whedon generate pdf |
PDF failed to compile for issue #2817 with the following error: Error reading bibliography file paper.bib: |
@whedon generate pdf |
As an update to the above:
We are working on benchmarking and addressing |
@johanneswerner I was able to reproduce the error you have been getting from It is indeed related to Thanks again for your patience! |
@johanneswerner we have provided a graphic below using the We have also uploaded Thank you! |
Dear @akrinos thank you for your updates. I think I checked the above checkboxes that are taken care of (if I forgot something, please let me know). Unfortunately, I still encountered errors with the minimum example (run.log) and some of the tests also throw errors on my virtual instance (tests.log). Could you please have a look at them? Thank you very much for your effort, especially the benchmarking is very interesting. |
Hmm, it looks like you're still getting the same error, which is most likely the cause for the failed tests as well (although I haven't looked carefully at each failure). Did you reinstall via If this continues to be an issue, I suppose we should move to another thread per the guidelines. Thanks for your persistence! |
My apologies @akrinos, I pulled the git repository for the tests, but I forgot to reinstall via Test dataset runs accordingly after reinstallation with |
Thank you @johanneswerner! Did you still have one failed test as above (checkbox in initial review)? With regard to the last two remaining checkboxes, for the analysis of prokaryotes, as mentioned in 729306579, we have one default database that includes prokaryotes, and generally users can curate their own datasets including prokaryotes, we have just tailored the tool to eukaryotes. As far as other software to compare ours too, one other tool I found was CCMetagen, published earlier this year. This tool identifies eukaryotes in metagenomic samples, but is not for metatranscriptomes and only uses the NCBI database. It might be useful to point out how our approach is different from this one, which also compares itself to MEGAN. If it helps, I could include both of these explanations in either the text or the documentation, whichever seems more helpful. I think other than that, everything from your review has been addressed. Thanks again! |
Thank you for your comprehensive review @johanneswerner - this is shaping up nicely. Pinging @jcmcnch - are you still able to review this submission? Please let us know either way ASAP |
Hi @will-rowe @akrinos sorry for not getting back to you both sooner with this. I have been busy until recently and had unsubscribed from notifications (because I was getting about a dozen notifications from JOSS daily from unrelated reviews - perhaps something can be done by JOSS to prevent this). I am back on the case now, and will provide my comments ASAP, by the end of this week at the latest. |
Hi @will-rowe, thanks and no problem! I couldn't figure out how to edit the author list before. I ended up having to modify it to be release 1.0.2b on Zenodo here; hopefully that's okay! |
@whedon set 10.5281/zenodo.4422091 as archive |
OK. 10.5281/zenodo.4422091 is the archive. |
@whedon set v1.0.2b as version |
OK. v1.0.2b is the version. |
@whedon accept |
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2016 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2016, then you can now move forward with accepting the submission by compiling again with the flag
|
|
@akrinos - could you please merge this PR before we proceed? |
I merged it in, thanks @arfon! |
@whedon accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2021 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2021, 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... |
@johanneswerner, @jcmcnch - many thanks for your reviews here and to @will-rowe for editing this submission. JOSS relies upon the volunteer efforts of folks like yourselves and we simply couldn't do this without you! ✨ @akrinos - 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:
|
Submitting author: @akrinos (Arianna Krinos)
Repository: https://github.com/AlexanderLabWHOI/EUKulele
Version: v1.0.2b
Editor: @will-rowe
Reviewer: @johanneswerner, @jcmcnch
Archive: 10.5281/zenodo.4422091
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
@johanneswerner & @jcmcnch, 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 @will-rowe 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 @johanneswerner
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @jcmcnch
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: