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]: EchemFEM: A Firedrake-based Python package for electrochemical transport #6531
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:
|
Software report:
Commit count by author:
|
Paper file info: 📄 Wordcount for ✅ The paper includes a |
License info: ✅ License found: |
|
@editorialbot commands |
Hello @tlroy, here are the things you can ask me to do:
|
@editorialbot check references |
|
Review checklist for @sudarshanv01Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @alizmaConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @TomTranterConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hello everyone! I see that the checklist are being filled, and issues are being opened in the repository. Thanks for all the work, do let me know if you have questions or need help. |
@TomTranter @sudarshanv01 @alizma. I think I answered all the issues that were opened. Please let me know if there is anything else to be done. |
All good for me |
Also fine for me! |
Thank you @TomTranter and @sudarshanv01 for completing the review. @alizma how is your review coming along? Please let me know if you need any support. |
Sorry for the delay, I've been a bit tied up with the end of this academic year. I hope to finish my review by the end of the week. |
I think the package satisfies all the technical requirements, but I'm a bit concerned with some of the examples failing and a lack of documentation for some of the solvers. @tlroy is there any chance you could take a look? |
No worries @alizma.
I see you have LLNL/echemfem#14 opened for the documentation. Would you mind opening issues for the examples failing, so that the authors/developers have actionable issues to work on? Thanks. |
Yes, I added the errors I ran into with examples to an existing issue about errors in the examples: LLNL/echemfem#10, which has some discussion about fixes. |
Thanks for pointing it out, I missed these additional comments on LLNL/echemfem#10. |
Submitting author: @tlroy (Thomas Roy)
Repository: https://github.com/LLNL/echemfem
Branch with paper.md (empty if default branch): joss
Version: v0.0.1
Editor: @RMeli
Reviewers: @sudarshanv01, @alizma, @TomTranter
Archive: Pending
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
@sudarshanv01 & @alizma & @TomTranter, 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 @RMeli 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 @sudarshanv01
📝 Checklist for @alizma
📝 Checklist for @TomTranter
The text was updated successfully, but these errors were encountered: