-
-
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]: Synthia: multi-dimensional synthetic data generation in Python #2863
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @khinsen, @mnarayan 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:
|
PDF failed to compile for issue #2863 with the following error: Can't find any papers to compile :-( |
@whedon generate pdf from branch joss-paper |
|
Hi all! 👋 Thank you so much, @khinsen, @mnarayan for accepting to review this. Please read the instructions above. Any questions, feedback on the paper, etc., please post here. Any very code-specific questions, suggestions, etc., please use the issues in the code repo and link to them from this thread so we can all keep track of them. 🌸 For an example of how this process plays out feel free to skim previous reviews, such as: #2285 and #2348. |
@whedon check references from branch joss-paper |
|
|
Question to @dmey, @tnagler, @letmaik: I am investigating authorship as required by JOSS reviewing guidelines. There is no doubt that @dmey is the main contributor to this project. Of the two other contributors, whose contributions look roughly equal in importance from the "size of commits" point of view, @tnagler is a co-author whereas @letmaik is merely acknowledged for "comments and contributions". If that's OK with all of you, it's fine with me as well of course, but it looks surprising. |
@khinsen That's fine, we discussed this offline in advance, no problem there. |
@khinsen it's good that you picked this up as it may appear a bit confusing -- all commits during development were squashed therefore the current metadata is not very indicative of the amount of work and contributions made during development. @tnagler made substantial contributions in the conceptual and practical development of the tool. @letmaik gave us suggestions and made contributions to the project. |
👋 @khinsen, please update us on how your review is going. |
👋 @mnarayan, please update us on how your review is going. |
@whedon Going fine, thanks for asking! |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
OK, I am being mean in talking like that to a bot, but I really don't know what I am expected to do in reply to the request for "updating us"! |
@khinsen the bot is just checking to see you are indeed doing the review, exactly like a reminder from a more "traditional" journal in your email inbox. Thus, you are able to ignore as you're already doing it. If you are curious what commands @whedon does accept, I am sure you can figure that out too — haha: https://joss.readthedocs.io/en/latest/whedon.html |
@oliviaguest The checklist asks "Do references in the text use the proper citation syntax?", referring to a description of Markdown syntax for citations. What I have for review is a PDF file. I don't quite understand what I am supposed to do. Should I be looking at some Markdown source instead? If so, where can I find it? |
@dmey After going through the checklist for the software, here comes my review of the paper itself. Overall, it looks very good, there is just one point I would like you to address: "Do the authors describe how this software compares to other commonly-used packages?" No. Please add a statement on this question, even if it is only "we are not aware of any other comparable software". The Wikipedia page on Synthetic Data lists one Open Source package (DataGenerator) and one publicly available package without a licence (Dataset Generator), in addition to proprietary software. |
@khinsen do you mean where is the paper being compiled from? Here: https://github.com/dmey/synthia/tree/joss-paper |
@oliviaguest Exactly. Thanks for the pointer, I hadn't considered looking for a specific branch. Perhaps the review instructions should contain that pointer, given that reviewers are expected to check the Markdown source. |
@oliviaguest Everything is obvious if you do it often enough! Authors and editors are a lot more familiar with @whedon than reviewers, and for now that has been my only role in JOSS. We can usually do our jobs without ever talking to @whedon. Which I think is great, reviewing for JOSS is a real pleasure because of the absence of technical boilerplate tasks. |
@danielskatz thanks -- sounds good with me. |
@danielskatz and @oliviaguest would you have an update about this submission? Please let me know if you are waiting for further changes from my side. If not please can you use #2863 (comment) as final version for the summary-paper and version 1.1.0 with corresponding Zenodo archive https://zenodo.org/record/5358432 (currently set to an older version). |
@whedon set v1.1.0 as version |
OK. v1.1.0 is the version. |
@whedon set 10.5281/zenodo.5358432 as archive |
OK. 10.5281/zenodo.5358432 is the archive. |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2602 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2602, then you can now move forward with accepting the submission by compiling again with the flag
|
@dmey can you check that invalid DOI please and the final proof? |
I see the version and archive are finished and attached, and I read through the paper and it looks good to me. Ready when we hear back from @oliviaguest's questions. |
@oliviaguest and @kthyng I am happy with the final proof, thank you. Re DOI showing as invalid this is correct but it has not yet been accepted (and deposited). The DOI will not change so I can (a) leave it as is, (b) update the reference to include 'under review', or (c) wait for the DOI to be deposited -- this should take a few weeks. Happy with any of those options. |
I'm fine with letting the DOI stand. What do you think @oliviaguest? |
I think we can proceed without hearing from @oliviaguest on this small issue. Seems fine as is. |
@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... |
Congrats on your new publication @dmey! Many thanks to editor @oliviaguest and reviewers @khinsen and @mnarayan for your time, hard work, and expertise! |
🎉🎉🎉 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 @kthyng and many thanks again to @oliviaguest @danielskatz @khinsen and @mnarayan for your help and feedback provided. |
@dmey you are welcome and glad to see this out! ✨ |
Submitting author: @dmey (D. Meyer)
Repository: https://github.com/dmey/synthia
Version: v1.1.0
Editor: @oliviaguest
Reviewer: @khinsen, @mnarayan
Archive: 10.5281/zenodo.5358432
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
@khinsen & @mnarayan, 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 @oliviaguest 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 @khinsen
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @mnarayan
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: