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]: Curifactory: A research experiment manager #5793
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 @abhishektiwariConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@WildfireXIII A few questions/improvements to meet JOSS guidelines Contribution and authorship:
Community guidelines:
Installation instructions:
State of the field:
Quality of writing: “reusability” not “reusablility” “This is often a result of” not “This often is a result of” This leads to the next core concept of Curifactory, which are "parameters”. not This leads to the next core concept of Curifactory, which are "parameters." Lastly, a suggestion for future versions of curifactory,
|
@editorialbot commands |
Hello @deniederhut, here are the things you can ask me to do:
|
Review checklist for @deniederhutConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Okay I've got some issues opened regarding some of the functionality in the getting started guide:
@kellyrowland as Nathan mentioned in the other ticket, a paper about this software was published at SciPy. Will that be a problem? |
@editorialbot generate pdf |
@abhishektiwari thanks for your review!
Jason's contributions have largely been internal, he has been invaluable throughout Curifactory's development in discussing/proposing features and ideas, making implementation suggestions, testing the tool, and code review.
Ahh yeah I forgot about that, I added a CONTRIBUTING.md to the project root
I added a requirements section to the README.md
Good point, we actually looked into Luigi a bit back in the beginning, I added a few sentences in the Statement of Need section comparing it to that.
I updated the paper with these corrections.
Yeah that's a really good idea, informally you can sort of already do this to some degree: running |
Thanks @WildfireXIII for making those revisions. Updated my checklist. |
@WildfireXIII bit of a meta-question for you -- there are a few things in the curifactory API design that feel unexpected to me. I'm not sure any of them are actually bugs, they could all just be my mental model not matching reality. What's the best way to give you that feedback? The issue template feels very focused on actual bugs so I've felt a bit bad about opening "I don't understand why it works this way" tickets there. |
@deniederhut no worries, feel free to still just open them as issues, for now I use issues pretty informally as my brainstorming, todo list, and design concept ideas I need to discuss with others, so you're welcome to just delete the issue template text and put whatever for the purposes of this review |
Just so it's here, I'll mention what I put in my message to the editors on the submission to clarify our intent, the paper we submitted to SciPy was meant more as a discussion of the "theory", a set of design aspects that can be used to compare/analyze experiment workflow tools (we did go on to spend a couple pages covering how Curifactory meets those), while this submission is meant to focus just on the Curifactory package itself and to get more of a formal external review of the tool. (It's also worth noting we've updated Curifactory pretty substantially since our SciPy work) |
I think this is okay based on the separation of the theory versus the software package itself, but I'll ask @danielskatz to weigh in here. |
@editorialbot generate pdf |
thanks for all of the work here, everyone. @deniederhut just checking in if you're able to take a look at this again with the latest updates. |
@kellyrowland a new release has been created (v0.16.1), and I've put it on zenodo: https://zenodo.org/records/10035496 (DOI: 10.5281/zenodo.10035496) @deniederhut and @abhishektiwari thanks so much for the reviews! I appreciate your all's time! |
@editorialbot set 10.5281/zenodo.10035496 as archive |
Done! archive is now 10.5281/zenodo.10035496 |
@editorialbot set v0.16.1 as version |
Done! version is now v0.16.1 |
@editorialbot check references |
|
@editorialbot generate pdf |
@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#4723, then you can now move forward with accepting the submission by compiling again with the command |
@WildfireXIII - I'm the track editor who will handle the rest of the process for your submission. I've suggested some minor changes in ORNL/curifactory#112 (adding the country to the affiliation, fixing capitalization to match JOSS's style, removing an extra period in a title). Please merge this, or let me know what you disagree with, then we can proceed to acceptance and publication. |
@danielskatz great, thanks! I merged your 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#4726, 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 @WildfireXIII (Nathan Martindale) and co-authors on your publication!! And thanks to @abhishektiwari and @deniederhut for reviewing, and to @kellyrowland 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:
|
Submitting author: @WildfireXIII (Nathan Martindale)
Repository: https://github.com/ORNL/curifactory
Branch with paper.md (empty if default branch):
Version: v0.16.1
Editor: @kellyrowland
Reviewers: @abhishektiwari, @deniederhut
Archive: 10.5281/zenodo.10035496
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
@abhishektiwari & @deniederhut, 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 @kellyrowland 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 @abhishektiwari
📝 Checklist for @deniederhut
The text was updated successfully, but these errors were encountered: