-
-
Notifications
You must be signed in to change notification settings - Fork 36
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]: sfctools - A toolbox for stock-flow consistent, agent-based models #4980
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 |
|
👋🏼 @fuadlabgit @alanlujan91 @npalmer-professional 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 @editorialbot generate my checklist as the top of a new comment in this thread. These checklists contain the JOSS requirements. As you go over the submission, please check any items that you feel have been satisfied. The first comment in this thread also contains links to the JOSS reviewer guidelines. The JOSS review is different from most other journals. Our goal is to work with the authors to help them meet our criteria instead of merely passing judgment on the submission. As such, the reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention #4980 so that a link is created to this thread (and I can keep an eye on what is happening). Please also feel free to comment and ask questions on this thread. In my experience, it is better to post comments/questions/suggestions as you come across them instead of waiting until you've reviewed the entire package. We aim for reviews to be completed within about 2-4 weeks. Please let me know if any of you require some more time. We can also use EditorialBot (our bot) to set automatic reminders if you know you'll be away for a known period of time. Please feel free to ping me (@sbenthall) if you have any questions/concerns. |
Review checklist for @alanlujan91Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Ping @npalmer-professional is there anything I can do to help you get started with your review? Please don't hesitate to reach out to me on this thread, or by email if you prefer. |
Dear @sbenthall and @npalmer-professional, belated happy new year and thank you so much for keeping the review process up. In the meantime, The code has been cleaned up and developed a bit further. Therefore, I have merged back the development branch 'spotykach' to the paper branch. This should make the review a little easier (slightly cleaner code) and also includes some more tests. In order to distinguish the originally submitted version, I have created a tag in the paper branch. The original version is joss-initial-submission and the new version is tagged joss-pre-review. Hope this helps you. In case the extent of the framework exceeds the capacities of the reviewer, I am also willing to discuss cutting parts off the repository for a more lightweight review. All the best, Thomas |
@editorialbot generate pdf |
@fuadlabgit I believe that it is just fine that you merged in progress during the review period, since reviews at JOSS are potentially interactive. It sounds like you've made some good improvements and unless I hear an objection from the reviewers, I'll trust that didn't create a problem in terms of scope. |
Review checklist for @npalmer-professionalConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Reproducibility/Functionality/Performance: https://gitlab.com/dlr-ve/esy/sfctools/framework/-/issues/15 Having a hard time reproducing the paper example to test out functionality and performance. |
State of the field: Brief discussion of other software (1 sentence). More in depth discussion might be needed. Quick search also brought up https://github.com/AB-CE/abce and https://github.com/salesforce/ai-economist. Are these unrelated? |
@editorialbot generate pdf |
2 similar comments
@alanlujan91 , thank you so much for reviewing this part! Indeed, it would be valuable to mention more frameworks. I have |
@alanlujan91 thank you for testing the example. Improvements have been made about the reproducability and description of the paper example. See the issue page in Gitlab and my answers therein. You might need to run pip install --upgrade sfctools to get the latest version before running the example file. I have replaced the numerical values with sympy expressions "x" and "d" in order to get a result which is not too confusing and which better reflects the paper example. However, symbolic expressions are only supported in the latest version of sfctools. |
@npalmer-professional Any updates on this? |
Dear @npalmer-professional, thank you so much for reviewing and for your interest in the project! I will try to give some answers / further explanations about the points left open (see comments below)
There is no data involved in the framework, as it represents a general modeling framework, rather than a specific calibrated model.
Please have a look at the responses to alanlujan91 from March 2023. The paper example should be running and replicating exactly the transactions-flow table given in the paper.
Here, I want to point out to the documentation page. There is a readthedocs page available here
We do not explicitly claim performance gains, but point out methodological gains from using framework vs. other frameworks/methods such as programming models in MS Excel (as sometimes done in the field).
Sfctools is registerd at Pypi, see https://pypi.org/project/sfctools/
The examples are placed under sfctools/examples and are also included in the API documentation: https://sfctools-framework.readthedocs.io/en/latest/doc_api_examples/examples_framework.html
An overview of the core modules is given here https://sfctools-framework.readthedocs.io/en/latest/doc_software/structure.html
The gitlab repository uses automated tests within its CI pipeline. You can check the pipeline history at https://gitlab.com/dlr-ve/esy/sfctools/framework/-/pipelines
The 'Context and Scope' page of the documentation aims at giving a glimpse into the state of the field, different perspectives of modeling etc. and also gives 14 references. Further, a table has been added to the paper to show a comparison of sfctools to other modeling frameworks used in the community.
|
|
|
👋 @openjournals/sbcs-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#4382, then you can now move forward with accepting the submission by compiling again with the command |
@editorialbot generate pdf |
Sorry, the link to the documentation page was cyan and not clickable -> now it works. |
Hmm. @fuadlabgit did you make a change to the repository? If so, I think I need to ask you to make another archive/DOI and release. Or have I misunderstood? |
@sbenthall so sorry for the confusion. I was not aware this was producing inconveniences. Here is the updated, final version I have created a tag https://gitlab.com/dlr-ve/esy/sfctools/framework/-/releases/joss_final |
@sbenthall is this now ready? 😊 |
@editorialbot set joss_final as version |
Done! version is now joss_final |
@editorialbot set 10.5281/zenodo.8118870 as archive |
Done! archive is now 10.5281/zenodo.8118870 |
@editorialbot recommend-accept |
|
|
@fuadlabgit It should be all good now. I've updated the submission. @oliviaguest I once again recommend this submission for acceptance! |
👋 @openjournals/sbcs-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#4387, then you can now move forward with accepting the submission by compiling again with the command |
@sbenthall @alanlujan91 @npalmer-professional Wonderful. Thank you so much again for your effort, for reviewing and for the extremely valuable feedback, especially about the examples! |
@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 @fuadlabgit! Huge thanks to @sbenthall and the reviewers @npalmer-professional, @alanlujan91. 🥳 👏 |
🎉🎉🎉 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: @fuadlabgit (Thomas Baldauf)
Repository: https://gitlab.com/dlr-ve/esy/sfctools/framework/
Branch with paper.md (empty if default branch): paper
Version: joss_final
Editor: @sbenthall
Reviewers: @npalmer-professional, @alanlujan91
Archive: 10.5281/zenodo.8118870
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
@npalmer-professional & @alanlujan91, 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 @sbenthall 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 @alanlujan91
📝 Checklist for @npalmer-professional
The text was updated successfully, but these errors were encountered: