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]: Aletheia: an open-source toolbox for steganalysis #5982
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 |
|
👋🏼 @daniellerch, @YassineYousfi, @ragibson, this is the review thread for the paper. All of our communications will happen here from now on. As a reviewer, the first step is to create a checklist for your review by entering
at the top of a new comment in this thread. There are additional guidelines in the first comment of this issue. Please don't hesitate to ping me (@mstimberg) if you have any questions/concerns. |
Review checklist for @ragibsonConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
(Apologies for originally pinging the wrong Daniel in this comment) Finally getting around to this. Just a quick note on the list of authors -- it seems the primary author @daniellerch wrote the software package and both authors collaborated on a few of the detection techniques used by the package (e.g., https://dl.acm.org/doi/10.1145/3335203.3335738 and https://ieeexplore.ieee.org/document/9722958), so the authors list looks good to me. There are a handful of other contributors on the GitHub repository, but they're all much more minor. |
A note on the data sharing point -- the paper basically contains no original data since the examples are generic and I was able to effectively run them with my own local images. Ditto on reproducibility of the examples in the paper. |
I would note that I ran into a few opaque errors in the simulators, but they look like research-grade code sourced from other authors/institutions (hence the requirement to accept secondary licenses and download the code from https://github.com/daniellerch/aletheia-external-resources). In these cases, the issues are external to the authors' work. @daniellerch That said, I would like to see the secondary repository explicitly linked in the paper and/or READMEs of the package rather than just in |
@editorialbot generate pdf |
@ragibson Many thanks for your review and comments so far. @YassineYousfi, did you have any chance to look at the software/paper yet? |
@mstimberg not yet, but I will check it out and review it this weekend! |
@mstimberg The second author didn't directly contribute code, yet their involvement has been recognized, and they are now included in the Zenodo archive. I believe everything is now in order. Thank you! |
@daniellerch Many thanks for the changes. From your changes on Zenodo, I just realized that the affiliations concern three separate institutions (even though I assume they are all co-located). Could you update the paper in that regard, i.e. list them as ¹²³ as in the Zenodo archive (stating "Barcelona, Spain" each time – assuming this is correct, of course)? Thanks! |
@editorialbot generate pdf |
@mstimberg Thank you for pointing this out. The changes have been made to the paper, listing the affiliations as ¹²³ and stating 'Barcelona, Spain' for each, as per your suggestion. Thanks again! |
@editorialbot check references |
|
@editorialbot recommend-accept All looking good from my side, handing things over to the topic editor for the final steps. Thanks again to everyone involved! |
|
|
👋 @openjournals/csism-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#4903, then you can now move forward with accepting the submission by compiling again with the command |
Thanks @mstimberg! @daniellerch - I'll now proofread this and let you know the next steps |
👋 @daniellerch - I've suggested some minor changes in the paper text in daniellerch/aletheia#34 - please merge this, or let me know what you disagree with, then we can proceed to acceptance and publication |
Thank you for your input @danielskatz, I've merged the suggested changes. |
@editorialbot recommend-accept |
|
|
👋 @openjournals/csism-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#4912, 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... |
Congratulations to @daniellerch (Daniel Lerch-Hostalot) and co-author on your publication!! And thanks to @YassineYousfi and @ragibson for reviewing, and to @mstimberg for editing! |
🎉🎉🎉 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:
|
Thanks @YassineYousfi, @ragibson, @mstimberg and @danielskatz for your great work on the JOSS publication. Really appreciate your insightful reviews and edits! |
Submitting author: @daniellerch (Daniel Lerch-Hostalot)
Repository: https://github.com/daniellerch/aletheia
Branch with paper.md (empty if default branch): master
Version: v0.3
Editor: @mstimberg
Reviewers: @YassineYousfi, @ragibson
Archive: 10.5281/zenodo.10497963
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
@YassineYousfi & @ragibson, 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 @mstimberg 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 @ragibson
📝 Checklist for @YassineYousfi
The text was updated successfully, but these errors were encountered: