-
-
Notifications
You must be signed in to change notification settings - Fork 37
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]: LiberTEM: Software platform for scalable multidimensional data processing in transmission electron microscopy #2006
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @alvarolopez, @mamcdona77 it looks like you're currently assigned to review 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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
@majensen I am done, the paper is ok to go in its current form. |
@alvarolopez - very much appreciated! |
Will update early next week
Michael
Michael A McDonald, MD, PhD
… On Jan 22, 2020, at 9:52 PM, Mark Jensen ***@***.***> wrote:
@alvarolopez - very much appreciated!
@mamcdona77 - how are things progressing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
@whedon generate pdf |
Would it be OK to include a small wording improvement in the first sentence? |
@uellue of course! |
@mamcdona77 - How is it coming along? Can I help in any way? Thanks |
OK, @fedorov is now a reviewer |
Thanks @fedorov for agreeing to review this work. |
@uellue very nice work! I submitted just one issue, and also have few clarification questions below, before I check off the remaining items in the checklist (am not sure if I should submit issues about those):
|
Hi @fedorov, thank you for the feedback! Good points, we'll work on it. |
@whedon remove @mamcdona77 as reviewer |
@labarba Is this a requirement that there's a 1:1 match with the Zenodo title? It seems that I overlooked it. The title of the Zenodo deposition is the default one if GitHub integration for releases is activated. We can also call the paper As @majensen wrote, we already have quite a number of releases on Zenodo and it would seem a bit odd to rename it just to match a paper title about it. |
We have a (soft) policy of asking authors to manually edit the metadata of the Zenodo deposit after the release has done its thing, so the titles and author list matches the paper. Of course we don't want you to change the title of the paper to match the repo name. Look, I'm not insisting. That's what we regularly do. If you have your reasons, do as you wish. |
@labarba Thank you for the clarification! If possible, I'd like to leave it as it is. Since both paper title and archive contain the unique name "LiberTEM", I think that readers will understand that the two are referring to each other even if the titles don't match 100%. :-) |
@labarba Sounds like we are ready to pull the trigger |
@majensen - when a submission is ready to be accepted, please tag @openjournals/joss-eics to be sure you get the on-duty AEiC, which changes week to week... |
@openjournals/joss-eics This work is recommended for publication. |
👋 @uellue - if you can merge LiberTEM/LiberTEM#823, we can finalize the publication |
@whedon accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#1507 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1507, then you can now move forward with accepting the submission by compiling again with the flag
|
👋 @uellue - sorry, I missed one - can you also merge LiberTEM/LiberTEM#824 |
@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... |
@danielskatz Thank you! 🥳 🎉 |
Thanks to @alvarolopez & @fedorov for reviewing! And congratulations to @uellue (Dieter Weber) and co-authors!! |
🎉🎉🎉 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:
|
Thank you @danielskatz @alvarolopez @fedorov @majensen for reviewing and managing the submission! @sk1p @kruzaeva @woozey @anandbaburajan @twentyse7en @jan-car @irahulcse @sayandip18 @magnunor Knut @rafaldb Many thanks and congratulations! |
As a general remark, open writing and open review in combination with GitHub worked really well IMO, and the whole process was a refreshing, pleasant and productive experience. I'll be sure to recommend JOSS for papers like this. 👍 |
Submitting author: @uellue (Dieter Weber)
Repository: https://github.com/LiberTEM/LiberTEM
Version: 0.5.0
Editor: @majensen
Reviewers: @alvarolopez, @fedorov
Archive: 10.5281/zenodo.3763313
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
@alvarolopez & @fedorov, 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 @majensen know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @alvarolopez
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @fedorov
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: