Closed editorialbot closed 10 months ago
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:
@editorialbot commands
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
@editorialbot generate pdf
Software report:
github.com/AlDanial/cloc v 1.88 T=0.05 s (1093.9 files/s, 202869.7 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Julia 34 635 270 7218
Markdown 6 200 0 532
YAML 7 2 10 221
TeX 1 14 0 132
TOML 2 5 0 34
-------------------------------------------------------------------------------
SUM: 50 856 280 8137
-------------------------------------------------------------------------------
gitinspector failed to run statistical information for the repository
Wordcount for paper.md
is 469
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/141000671 is OK
- 10.3390/a13050110 is OK
- 10.1046/j.1365-8711.2000.03086.x is OK
- 10.1145/3240765.3240860 is OK
- 10.5194/gmd-15-3161-2022 is OK
- 10.1016/j.jhydrol.2015.07.036 is OK
- 10.5334/jors.431 is OK
- 10.1002/nla.2281 is OK
- 10.1155/asp/2006/31062 is OK
- 10.1038/s41598-022-26898-1 is OK
MISSING DOIs
- None
INVALID DOIs
- None
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@ziyiyin97, @shahmoradi
This is the review thread. Firstly, type
@editorialbot generate my checklist
to generate your own checklist. In that checklist, there are many check items. Whenever you complete the corresponding task, you can check off them.
Please write your comments as separate posts and do not modify your checklist descriptions.
The review process is interactive so you can always interact with the authors, reviewers, and the editor. You can also create issues and pull requests in the target repository. Please do mention this thread's URL in the issues so we can keep tracking what is going on out of our world.
Please do not hesitate to ask me about anything, anytime.
Thank you in advance!
@shahmoradi - Could you please generate your checklist before starting your review? Thank you in advance.
Dear @ziyiyin97, @shahmoradi
Could you please update your status and inform us on how is your review going?
Thank you in advance!
Hi, sorry for the delay. I should be able to finish my reviews in a week if that works.
@ziyiyin97 - thank you for the status update
I just posted my reviews via an issue to the repository shown above.
@ziyiyin97 - thank you for opening the issues and minor suggestions.
@PieterjanRobbe - could you please perform the changes indicated in the issue?
Hi @ziyiyin97, thank you so much for your time reviewing the software and paper! I've addressed your comments with this commit.
LGTM!
@ziyiyin97 - Thank you, could you please take a look at your task list and complete filling boxes if they are okay.
@shahmoradi - Do we have a chance to get a status update? Could you please update your status and review? Thank you in advance
I apologize for the late response. I have been dealing with multiple deadlines and trips over the past month. I am still on a trip, but I will try to complete this review within the next few days.
@shahmoradi - thank you for the response. We are looking forward to hear from you.
Thank you for your work @PieterjanRobbe . I have reviewed this paper, and most look fine and justified. Two minor issues:
Performance
, I have marked all other checklist items as complete.@shahmoradi - thank you for your valuable review and suggestions.
@PieterjanRobbe - could you please apply the suggestions? thank you in advance.
Hi @shahmoradi, thank you so much for your time and effort! Here's an answer to your questions:
Kernel()
and Generator()
are supposed to be replaced by a call to the appropriate methods, as mentioned below the code block. I've tested all other code blocks in the tutorial, and they seem to work fine for me.Thanks again!
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@shahmoradi - could you please review the latest changes? thank you in advance!
I will try to finalize this review tonight. Thanks for your patience.
On Mon, Aug 14, 2023 at 3:20 PM Mehmet Hakan Satman < @.***> wrote:
@shahmoradi https://github.com/shahmoradi - could you please review the latest changes? thank you in advance!
— Reply to this email directly, view it on GitHub https://github.com/openjournals/joss-reviews/issues/5595#issuecomment-1678001942, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDXNCNWLLCXL2ZK6O2TN3LXVKB7HANCNFSM6AAAAAAZVJWIYM . You are receiving this because you were mentioned.Message ID: @.***>
Hi @shahmoradi, thanks again for your comments! Please let me know if there's anything I can do to help finalize your review.
Dear @shahmoradi
It seems there is a single unchecked item in your task list. Please review it and tell us your latest decisions.
Thank you in advance.
Thanks, Mehmet, for your reminder. I am traveling at the moment. Please allow me to respond by Thursday. Thanks for your patience, Amir
On Tue, Aug 29, 2023 at 11:32 AM Mehmet Hakan Satman < @.***> wrote:
Dear @shahmoradi https://github.com/shahmoradi
It seems there is a single unchecked item in your task list. Please review it and tell us your latest decisions.
Thank you in advance.
— Reply to this email directly, view it on GitHub https://github.com/openjournals/joss-reviews/issues/5595#issuecomment-1697786368, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDXNCIURXTSZMSXUTEEFL3XXYKRBANCNFSM6AAAAAAZVJWIYM . You are receiving this because you were mentioned.Message ID: @.***>
@shahmoradi - Thank you for the response, sorry for bothering and being verbose in your vacation season. We are looking forward to hearing from you.
Thank you for your response @PieterjanRobbe. My checklist is now complete.
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/141000671 is OK
- 10.3390/a13050110 is OK
- 10.1046/j.1365-8711.2000.03086.x is OK
- 10.1145/3240765.3240860 is OK
- 10.5194/gmd-15-3161-2022 is OK
- 10.1016/j.jhydrol.2015.07.036 is OK
- 10.5334/jors.431 is OK
- 10.1002/nla.2281 is OK
- 10.1155/asp/2006/31062 is OK
- 10.1038/s41598-022-26898-1 is OK
MISSING DOIs
- 10.1016/s0098-3004(00)00063-7 may be a valid DOI for title: Geostatistics: modeling spatial uncertainty
INVALID DOIs
- None
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@editorialbot set <DOI here> as archive
@editorialbot set <version here> as version
@editorialbot generate pdf
@editorialbot check references
and ask author(s) to update as needed@editorialbot recommend-accept
@PieterjanRobbe - I've just sent a pull request that contains minor changes in your bibtex. Could you please review the changes and apply them if they are okay?
PR: https://github.com/PieterjanRobbe/GaussianRandomFields.jl/pull/51
Thank you in advance.
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/141000671 is OK
- 10.3390/a13050110 is OK
- 10.1046/j.1365-8711.2000.03086.x is OK
- 10.1016/s0098-3004(00)00063-7 is OK
- 10.1145/3240765.3240860 is OK
- 10.5194/gmd-15-3161-2022 is OK
- 10.1016/j.jhydrol.2015.07.036 is OK
- 10.5334/jors.431 is OK
- 10.1002/nla.2281 is OK
- 10.1155/asp/2006/31062 is OK
- 10.1038/s41598-022-26898-1 is OK
MISSING DOIs
- None
INVALID DOIs
- None
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@PieterjanRobbe - One more PR about bibtex
PR: https://github.com/PieterjanRobbe/GaussianRandomFields.jl/pull/52
@PieterjanRobbe - Meanwhile checking out the references and the manuscript, you can follow the instructions given in the https://github.com/openjournals/joss-reviews/issues/5595#issuecomment-1701418760. We need a tagged release created in the repository and then a related Zenodo archive DOI which is linked to this tagged release. The Zenodo archive should match the exact title of the paper, the author list and their information including ORCIDs.
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1137/141000671 is OK
- 10.3390/a13050110 is OK
- 10.1046/j.1365-8711.2000.03086.x is OK
- 10.1016/s0098-3004(00)00063-7 is OK
- 10.1145/3240765.3240860 is OK
- 10.5194/gmd-15-3161-2022 is OK
- 10.1016/j.jhydrol.2015.07.036 is OK
- 10.5334/jors.431 is OK
- 10.1002/nla.2281 is OK
- 10.1155/asp/2006/31062 is OK
- 10.1038/s41598-022-26898-1 is OK
MISSING DOIs
- None
INVALID DOIs
- None
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@PieterjanRobbe - The manuscript and the bibtex look good to me.
@editorialbot set v2.2.3 as version
Done! version is now v2.2.3
Submitting author: !--author-handle-->@PieterjanRobbe<!--end-author-handle-- (Pieterjan Robbe) Repository: https://github.com/PieterjanRobbe/GaussianRandomFields.jl Branch with paper.md (empty if default branch): joss Version: v2.2.4 Editor: !--editor-->@jbytecode<!--end-editor-- Reviewers: @ziyiyin97, @shahmoradi Archive: 10.5281/zenodo.8306255
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
@ziyiyin97 & @shahmoradi, 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 @jbytecode 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 @ziyiyin97
📝 Checklist for @shahmoradi