-
-
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]: svFSI: A Multiphysics Package for Integrated Cardiac Modeling #4118
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @chennachaos, @JaroslavHron, @ @axel-loewe 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:
|
Wordcount for |
|
PDF failed to compile for issue #4118 with the following error:
|
|
@whedon re-invite @axel-loewe as reviewer |
OK, the reviewer has been re-invited. @axel-loewe please accept the invite by clicking this link: https://github.com/openjournals/joss-reviews/invitations |
@axel-loewe apologies for the second invite. I had added you initially as |
@chennachaos, @JaroslavHron, @axel-loewe thanks for your help with this review!!!! 🥳 You may now start. Let me know if you have any questions. |
@whedon generate pdf |
Hi @Kevin-Mattheus-Moerman - it seems that i have missed the invitation, and its now expired - could you reinvite me. |
@whedon re-invite @JaroslavHron a reviewer |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
Zhu et al. present svFSI, a simulation software for cardiac physiology. It covers multiple cardiac functions (physics): solid mechanics, hemodynamics and apparently cardiac electrophysiology. The effort of developing a software for this challenging coupled problem and making if publicly available is appreciated.
While there is no
From the commit history, this does not seem to be the case (3 minor commits only). I am aware that the public commits may not be representative of the entire history of the project. Please comment on it.
I did not manage to compile the code with clang 13.0 (Undefined identifiers in METISLib).
svFSI is introduced as a multi-physics simulator including electrophysiology, tissue mechanics and blood dynamics. Unfortunately, I could not find any information on electrophysiology in the paper or the documentation (http://simvascular.github.io/docssvFSI.html). As such, I suggest to either document these features or to remove these aspects from the paper.
The website lists them but there is no automated way of installing them. Later on, I discovered more detailed information in the
As coupling multiple physics is a complex problem and also multiplies the number of user-chosen parameters, types of boundary conditions, pre- and postprocessing steps etc., I assume that svFSI probably provides only a subset of features of established single-physics simulators. I believe it would be important for potential new users to learn about which features are supported.
Could not find such information. There are no CI pipelines implemented as GitHub actions. Cannot judge on external CI.
Could not find such information in the repository or the svFSI part of the SimVascular webpage.
Line 27: Please change the reference to openCARP to this one: https://dx.doi.org/10.1016/j.cmpb.2021.106223
|
@Kevin-Mattheus-Moerman - it seems that there was some typo in the re-invitation and it failed |
@whedon re-invite @JaroslavHron as reviewer |
OK, the reviewer has been re-invited. @JaroslavHron please accept the invite by clicking this link: https://github.com/openjournals/joss-reviews/invitations |
👋 @JaroslavHron, please update us on how your review is going (this is an automated reminder). |
👋 @chennachaos, please update us on how your review is going (this is an automated reminder). |
@editorialbot recommend accept |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
@editorialbot recommend-accept |
|
|
Element doi: [facet 'pattern'] The value 'https://doi.org/10.1016/j.cmpb.2021.106223' is not accepted by the pattern '10\.[0-9]{4,9}/.{1,200}'. |
@CZHU20 this PR should fix these issues with the DOIs: SimVascular/svFSI#92 |
@Kevin-Mattheus-Moerman this pull request has been merged. Thank you. |
@editorialbot check references |
|
@CZHU20 one more pull request: SimVascular/svFSI#93 to fix that missing DOI |
@Kevin-Mattheus-Moerman Done. |
@editorialbot check references |
|
@editorialbot recommend-accept |
|
|
👋 @openjournals/joss-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#3598, then you can now move forward with accepting the submission by compiling again with the command |
@editorialbot accept |
|
Congratulations @CZHU20 on this publication!!!! @chennachaos, @JaroslavHron and @axel-loewe thank you so much for your fantastic review efforts!!!!!! |
🐦🐦🐦 👉 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... |
Thank you all for your help! @Kevin-Mattheus-Moerman @chennachaos @JaroslavHron @axel-loewe |
🎉🎉🎉 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: @CZHU20 (Chi Zhu)
Repository: https://github.com/SimVascular/svFSI
Branch with paper.md (empty if default branch):
Version: 2022.09.26
Editor: @Kevin-Mattheus-Moerman
Reviewers: @chennachaos, @JaroslavHron, @axel-loewe
Archive: 10.5281/zenodo.7113485
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
@chennachaos & @JaroslavHron & @ @axel-loewe, 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 @Kevin-Mattheus-Moerman 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 @JaroslavHron
✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @axel-loewe
✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: