-
-
Notifications
You must be signed in to change notification settings - Fork 39
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]: hp3D: A Scalable MPI/OpenMP hp-Adaptive Finite Element Software Library for Complex Multiphysics Applications #5946
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 |
|
Hi, @peterrum @likask 👋 Welcome to JOSS and thanks for agreeing to review! The comments from @editorialbot above outline the review process, which takes place in this thread (possibly with issues filed in the hp3D repository). I'll be watching this thread if you have any questions. 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 this issue 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 a month or two. Please let me know if you require some more time. We can also use editorialbot to set automatic reminders if you know you'll be away for a known period of time. Please feel free to ping me (@jedbrown) if you have any questions/concerns. |
Review checklist for @peterrumConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @likaskConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@stefanhenneking I went through the checklist and read the paper. I didn't have any problems with installation after I have installed flex, bison, and libx11-dev; however, there are a bunch of warnings. I have run The text of the manuscript is overall clear and well written. I only have a few points:
I found the limitation to 3D a bit irritating when reading the text. In the documentation, it says there is a 2D version, however, it is not public. Could you add the same comment in the manuscript. I hope both codes share functionalities?
Do you have any numbers or papers that verify this claim?
I have tried the code on CPU. Is it also working on GPUs? If not, could you be more precise by stating
I would suggest to relax this statement about other libraries. For instance, deal.II supports parallel matrix-based and matrix-free hp-adaptivity (with threading via TBB and MPI; see Bangerth, Kayser-Herold 2009 and Fehling, Bangerth 2023). "anisotropic refinement" and additional ways to apply constrains seems to be features that sets hp3D apart. |
@stefanhenneking I have yet to test the code; however, I was able to install the code. The process will be much easier if you would add HP3D to the package manager, e.g. spack.pm —an easy job to do, which will make life easier. |
@peterrum I guessed where code dump paraview files, see folder hp3d/trunk/problems/POISSON/outputs/paraview |
@likask Thanks! Indeed! This is not the place one would start to look^^ |
@peterrum @likask Thank you both very much for agreeing to review our submission and for your feedback on the paper and the code. We really appreciate it! @peterrum thanks for pointing out the issue with the ParaView path. I'll add the output path to be printed on the screen by default. As you intuited correctly, Yes, we do have a 2D version that shares most of the functionality with the 3D version. So far the 2D version hasn't been made public because it'd need some refactoring and additional documentation first. We would like to either make that 2D version public in the future or perhaps extend the 3D version to support 2D computations directly. I will add a comment along those lines in the paper. Currently, only CPU-based architectures are supported. Thanks for bringing that up, I will change the sentence in the paper accordingly. The efficiency of the code is documented for fairly large problems (~O(10^9) dofs) in a few places - two papers are cited a bit further down in the paper in the "Statement of need" section. I'll also relax the statement about other libraries in the way you suggested. @likask I'm glad to hear you were able to install the library. Thank you for your suggestion to add hp3D to spack. I don't have any prior experience with that but I'll look into adding that to simplify installing hp3D and its dependencies. |
@editorialbot generate pdf |
@editorialbot generate pdf |
Hi, thanks for your work here. The paper reads well. I have two suggestions:
|
Thanks for your suggestions, @jedbrown. I completely agree that both CI and simplifying installation are important missing pieces. I will try to work on it as soon as possible and link the corresponding Issues / PRs to this thread here. |
@editorialbot generate pdf |
I updated the paper as you suggested, tagged the current release version (tagged The Zenodo DOI for the hp3D release The all-versions DOI is 10.5281/zenodo.10763374. I also added the Zenodo archive with DOI as a reference to the paper. |
@editorialbot set v1.0 as version |
Done! version is now v1.0 |
@editorialbot set 10.5281/zenodo.10763375 as archive |
Done! archive is now 10.5281/zenodo.10763375 |
@editorialbot recommend-accept |
|
|
👋 @openjournals/pe-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#5086, 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 @stefanhenneking on your article's publication in JOSS! Many thanks to @peterrum and @likask for reviewing this, and @jedbrown 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: @stefanhenneking (Stefan Henneking)
Repository: https://github.com/Oden-EAG/hp3d
Branch with paper.md (empty if default branch): joss-paper
Version: v1.0
Editor: @jedbrown
Reviewers: @peterrum, @likask
Archive: 10.5281/zenodo.10763375
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
@peterrum & @likask, 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 @jedbrown 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 @peterrum
📝 Checklist for @likask
The text was updated successfully, but these errors were encountered: