-
-
Notifications
You must be signed in to change notification settings - Fork 38
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]: checkpoint_schedules: schedules for incremental of adjoint simulations #6148
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 @matt-grahamConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@editorialbot generate pdf |
@matt-graham how is your review going? |
hi @KYANJO please let me know if you have questions about your first review? |
It’s going well so far about to finish
With utmost gratitude,
Brian KYANJO (He / Him / His),
President. SIAM Boise State Chapter,
VP. NSBE Boise State Society,
NSBE Public Policy SIG STEM Education, Team Co-Lead,
BSc. Physics and Mathematics | MSc. Mathematical Sciences | Computing Ph.D.
BSU,
https://github.com/KYANJO
…On Fri, Jan 5, 2024 at 2:26 AM Patrick Diehl ***@***.***> wrote:
@matt-graham <https://github.com/matt-graham> how is your review going?
—
Reply to this email directly, view it on GitHub
<#6148 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANLXYJDG64UCOI5O7YRWKC3YM7BLZAVCNFSM6AAAAABAVBTGYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZYGM3DOMZSGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
DISCLAIMER: The contents of this email and any attachments are
confidential. They are intended for the named recipient(s) only. If you
have received this email by mistake, please notify the sender immediately
and you are herewith notified that the contents are legally privileged and
that you do not have permission to disclose the contents to anyone, make
copies thereof, retain or distribute or act upon it by any means,
electronically, digitally or in print. The views expressed in this
communication may be of a personal nature and not be representative of
AIMS-Rwanda and/or any of its Initiatives.
|
I will let you know, if i get any questions
With utmost gratitude,
Brian KYANJO (He / Him / His),
President. SIAM Boise State Chapter,
VP. NSBE Boise State Society,
NSBE Public Policy SIG STEM Education, Team Co-Lead,
BSc. Physics and Mathematics | MSc. Mathematical Sciences | Computing Ph.D.
BSU,
https://github.com/KYANJO
…On Fri, Jan 5, 2024 at 2:27 AM Brian Kyanjo ***@***.***> wrote:
It’s going well so far about to finish
With utmost gratitude,
Brian KYANJO (He / Him / His),
President. SIAM Boise State Chapter,
VP. NSBE Boise State Society,
NSBE Public Policy SIG STEM Education, Team Co-Lead,
BSc. Physics and Mathematics | MSc. Mathematical Sciences | Computing
Ph.D. BSU,
https://github.com/KYANJO
On Fri, Jan 5, 2024 at 2:26 AM Patrick Diehl ***@***.***>
wrote:
> @matt-graham <https://github.com/matt-graham> how is your review going?
>
> —
> Reply to this email directly, view it on GitHub
> <#6148 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ANLXYJDG64UCOI5O7YRWKC3YM7BLZAVCNFSM6AAAAABAVBTGYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZYGM3DOMZSGE>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
--
DISCLAIMER: The contents of this email and any attachments are
confidential. They are intended for the named recipient(s) only. If you
have received this email by mistake, please notify the sender immediately
and you are herewith notified that the contents are legally privileged and
that you do not have permission to disclose the contents to anyone, make
copies thereof, retain or distribute or act upon it by any means,
electronically, digitally or in print. The views expressed in this
communication may be of a personal nature and not be representative of
AIMS-Rwanda and/or any of its Initiatives.
|
Review checklist for @KYANJOConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
I am through with my first review, and I have included comments in the pdf it's self. Find attached. PS; If there is a better way i can upload the document please let me know, Is it better to create a pull request instead? |
@Ig-dolci please take a look at the comments. |
@diehlpk Yes. I am aware. |
@Ig-dolci how are your changes going? |
@diehlpk It is going well. Hopefully, I will return with the reviewer's responses in the coming week. |
@Ig-dolci how are your changes going? |
1 similar comment
@Ig-dolci how are your changes going? |
@diehlpk It is going well. Unfortunately, it has required more time due to personal unforeseen the last week. |
@Ig-dolci When do you think you can have the new revision done? |
1 similar comment
@Ig-dolci When do you think you can have the new revision done? |
@Ig-dolci Please finish these tasks, so we can proceed with the paper
|
@diehlpk below the latest release and Zenodo DOI. |
@editorialbot set v1.0.3 as version |
Done! version is now v1.0.3 |
@editorialbot set 10.5281/zenodo.10817312 as archive |
Done! archive is now 10.5281/zenodo.10817312 |
@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#5132, then you can now move forward with accepting the submission by compiling again with the command |
@Ig-dolci - I'm the track editor who will finish the processing of this submission. I've suggested some small changes in firedrakeproject/checkpoint_schedules#65. Please merge this, or let me know what you disagree with. In addition, please add countries to all of the affiliations, though we don't need mailing address / postal codes. Usually, institution & country is enough, though department or similar is also fine. Also in the paper, "in the frame of DASH" in the acknowledgements sounds awkward to me, but I'm not sure how to fix it, as I don't really know what this means. Finally, in the references, the one for Pringle seems to be missing information about how to access it. Should it have a URL or something else? |
@danielskatz Your pull request PR 65 has been merged. In addition, I have opened PR 66, which includes the countries of all affiliations and a URL was added in the Pringle reference. Could you please verify if it sounds good for the final version of the JOSS paper? The mention of "in the frame of DASH" in the acknowledgements section refers to a funding project. I verified this terminology is usual in other papers authored by one of our authors. So, we should keep it in this form. |
Thanks - this looks good, though I'm not sure if the type for Pringle should be article or misc. We can try this and see how the PDF looks. |
I think misc is better for this case. I just changed from article to misc. |
@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#5162, 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 @Ig-dolci (Daiane Iglesia Dolci) and co-authors on your publication!! And thanks to @matt-graham and @KYANJO for reviewing, and to @diehlpk 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: @Ig-dolci (Daiane Iglesia Dolci)
Repository: https://github.com/firedrakeproject/checkpoint_schedules
Branch with paper.md (empty if default branch): main
Version: v1.0.3
Editor: @diehlpk
Reviewers: @matt-graham, @KYANJO
Archive: 10.5281/zenodo.10817312
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
@matt-graham, 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 @diehlpk 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 @matt-graham
📝 Checklist for @KYANJO
📝 Checklist for @KYANJO
The text was updated successfully, but these errors were encountered: