Closed editorialbot closed 1 week 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.90 T=0.03 s (3100.3 files/s, 167245.8 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
R 42 196 602 1181
C/C++ Header 32 120 301 664
XML 1 0 2 441
Markdown 4 98 0 283
Rmd 2 72 260 215
TeX 2 31 8 148
C++ 4 12 9 134
YAML 2 11 6 55
CSV 1 0 0 6
-------------------------------------------------------------------------------
SUM: 90 540 1188 3127
-------------------------------------------------------------------------------
Commit count by author:
53 mlysy
18 EFAcar
6 Elif Fidan Acar
1 alank2004
Paper file info:
π Wordcount for paper.md
is 1625
β
The paper includes a Statement of need
section
License info:
π‘ License found: GNU General Public License v3.0
(Check here for OSI approval)
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
ππΌ @EFAcar, @AlexisDerumigny, @mingzehuang this is the review thread for the paper. All of our communications will happen here from now on.
@AlexisDerumigny, @mingzehuang, as reviewers, the first step is to create a checklist for your review by entering
@editorialbot generate my checklist
as the top of a new comment in this thread.
These checklists contain the JOSS requirements. As you go over the submission, please check any items that you feel have been satisfied. The first comment in this thread also contains links to the JOSS reviewer guidelines.
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 openjournals/joss-reviews#6744
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 about 2-4 weeks. Please let me know if any of you require some more time. We can also use EditorialBot (our bot) to set automatic reminders if you know you'll be away for a known period of time.
Please feel free to ping me (@lrnv) if you have any questions/concerns.
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1201/9781315370279 is OK
- 10.18637/jss.v070.i05 is OK
- 10.1016/j.ecosta.2019.03.002 is OK
- 10.1111/j.1541-0420.2010.01472.x is OK
- 10.1214/13-EJS866 is OK
- 10.18637/jss.v021.i04 is OK
- 10.18637/jss.v034.i09 is OK
- 10.18637/jss.v039.i09 is OK
MISSING DOIs
- No DOI given, and none found for title: LocalCop: LocalCop: Local Likelihood Inference for...
- No DOI given, and none found for title: copula: Multivariate Dependence with Copulas
- No DOI given, and none found for title: VineCopula: Statistical Inference of Vine Copulas
- No DOI given, and none found for title: gamCopula: Generalized Additive Models for Bivaria...
- No DOI given, and none found for title: CondCopulas: Estimation and Inference for Conditio...
INVALID DOIs
- None
π @AlexisDerumigny & @mingzehuang How is it going with this review ? If you want/need, I could setup automatic reminders for you on this thread in like 1 week or 2. We usually aim at a 6-to-8 week reviewing time so you still have room ofc
Thank you for your reminder, @lrnv ! It goes smoothly. And I can wrap it up very soon once the author addressed the issues I raised :)
@AlexisDerumigny gentle bump :)
Hi, @lrnv. I've finished the review, and the author has addressed my concerns. I recommend accepting this paper :)
@AlexisDerumigny gentle bump :)
Thanks for the reminder @lrnv ! I am sorry for being so late, I had some personal problems. Now, the review is nearly completed up to two issues that I have opened.
Hi @lrnv, all my comments have been answered and I recommend the paper to be accepted.
@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 check references
@editorialbot check repository
Software report:
github.com/AlDanial/cloc v 1.90 T=0.03 s (3076.8 files/s, 169465.1 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
R 42 198 608 1225
C/C++ Header 32 120 305 680
XML 1 0 2 441
Markdown 4 111 0 296
Rmd 2 75 261 215
TeX 2 31 8 148
C++ 4 12 9 134
YAML 2 11 6 55
CSV 1 0 0 6
-------------------------------------------------------------------------------
SUM: 90 558 1199 3200
-------------------------------------------------------------------------------
Commit count by author:
57 mlysy
21 EFAcar
6 Elif Fidan Acar
1 alank2004
Paper file info:
π Wordcount for paper.md
is 1625
β
The paper includes a Statement of need
section
License info:
π‘ License found: GNU General Public License v3.0
(Check here for OSI approval)
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1201/9781315370279 is OK
- 10.18637/jss.v070.i05 is OK
- 10.1016/j.ecosta.2019.03.002 is OK
- 10.1111/j.1541-0420.2010.01472.x is OK
- 10.1214/13-EJS866 is OK
- 10.18637/jss.v021.i04 is OK
- 10.18637/jss.v034.i09 is OK
- 10.18637/jss.v039.i09 is OK
π‘ SKIP DOIs
- No DOI given, and none found for title: LocalCop: LocalCop: Local Likelihood Inference for...
- No DOI given, and none found for title: copula: Multivariate Dependence with Copulas
β MISSING DOIs
- 10.32614/cran.package.vinecopula may be a valid DOI for title: VineCopula: Statistical Inference of Vine Copulas
- 10.32614/cran.package.gamcopula may be a valid DOI for title: gamCopula: Generalized Additive Models for Bivaria...
- 10.32614/cran.package.condcopulas may be a valid DOI for title: CondCopulas: Estimation and Inference for Conditio...
β INVALID DOIs
- None
@EFAcar congratulations, we are reaching the end of this review. Would you please fix de missing DOI's before we can go to the post-review process ?
Thank you @lrnv. We've added DOIs for all CRAN packages (including the skipped ones) above.
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1201/9781315370279 is OK
- 10.18637/jss.v070.i05 is OK
- 10.1016/j.ecosta.2019.03.002 is OK
- 10.1111/j.1541-0420.2010.01472.x is OK
- 10.1214/13-EJS866 is OK
- 10.18637/jss.v021.i04 is OK
- 10.18637/jss.v034.i09 is OK
- 10.18637/jss.v039.i09 is OK
π‘ SKIP DOIs
- No DOI given, and none found for title: LocalCop: LocalCop: Local Likelihood Inference for...
- No DOI given, and none found for title: copula: Multivariate Dependence with Copulas
β MISSING DOIs
- 10.32614/cran.package.vinecopula may be a valid DOI for title: VineCopula: Statistical Inference of Vine Copulas
- 10.32614/cran.package.gamcopula may be a valid DOI for title: gamCopula: Generalized Additive Models for Bivaria...
- 10.32614/cran.package.condcopulas may be a valid DOI for title: CondCopulas: Estimation and Inference for Conditio...
β INVALID DOIs
- None
@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
@EFAcar Could you take care of your side of the (upper) post-review check list ?
@lrnv, @mingzehuang, @AlexisDerumigny thank you for the helpful review comments which have much improved our submission. Please find the post-review check list below:
mlysy/joss
after the post-review checks began. The most recent manuscript file is vignettes/joss/paper.md
on that branch.@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@EFAcar Everytthing looks good, except that the Zenodo archive's title and the paper title must be the same, could you change one of them so that they match ?
@editorialbot set 10.5281/zenodo.13801203 as archive
Done! archive is now 10.5281/zenodo.13801203
@editorialbot set v0.0.2 as version
Done! version is now v0.0.2
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1201/9781315370279 is OK
- 10.18637/jss.v070.i05 is OK
- 10.1016/j.ecosta.2019.03.002 is OK
- 10.1111/j.1541-0420.2010.01472.x is OK
- 10.1214/13-EJS866 is OK
- 10.32614/CRAN.package.LocalCop is OK
- 10.32614/CRAN.package.copula is OK
- 10.18637/jss.v021.i04 is OK
- 10.18637/jss.v034.i09 is OK
- 10.18637/jss.v039.i09 is OK
- 10.32614/CRAN.package.VineCopula is OK
- 10.32614/CRAN.package.gamCopula is OK
- 10.32614/CRAN.package.CondCopulas is OK
π‘ SKIP DOIs
- None
β MISSING DOIs
- None
β INVALID DOIs
- None
@EFAcar Everytthing looks good, except that the Zenodo archive's title and the paper title must be the same, could you change one of them so that they match ?
@lrnv the title for the Zenodo archive is now corrected.
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1201/9781315370279 is OK
- 10.18637/jss.v070.i05 is OK
- 10.1016/j.ecosta.2019.03.002 is OK
- 10.1111/j.1541-0420.2010.01472.x is OK
- 10.1214/13-EJS866 is OK
- 10.32614/CRAN.package.LocalCop is OK
- 10.32614/CRAN.package.copula is OK
- 10.18637/jss.v021.i04 is OK
- 10.18637/jss.v034.i09 is OK
- 10.18637/jss.v039.i09 is OK
- 10.32614/CRAN.package.VineCopula is OK
- 10.32614/CRAN.package.gamCopula is OK
- 10.32614/CRAN.package.CondCopulas is OK
π‘ SKIP DOIs
- None
β MISSING DOIs
- None
β INVALID DOIs
- None
:warning: Error preparing paper acceptance. The generated XML metadata file is invalid.
No declaration for attribute displaystyle of element mfrac
No declaration for attribute displaystyle of element mfrac
@EFAcar can you fix this?
@lrnv based on this review the error code seems to be due to using displaymode with \dfrac. We now changed these to \frac. Hope this will fix the issue.
@EFAcar ha yes it is true that you cant re-launch the process yourself. Let me check if it did it
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
β
OK DOIs
- 10.1201/9781315370279 is OK
- 10.18637/jss.v070.i05 is OK
- 10.1016/j.ecosta.2019.03.002 is OK
- 10.1111/j.1541-0420.2010.01472.x is OK
- 10.1214/13-EJS866 is OK
- 10.32614/CRAN.package.LocalCop is OK
- 10.32614/CRAN.package.copula is OK
- 10.18637/jss.v021.i04 is OK
- 10.18637/jss.v034.i09 is OK
- 10.18637/jss.v039.i09 is OK
- 10.32614/CRAN.package.VineCopula is OK
- 10.32614/CRAN.package.gamCopula is OK
- 10.32614/CRAN.package.CondCopulas is OK
π‘ SKIP DOIs
- None
β MISSING DOIs
- None
β INVALID DOIs
- None
Submitting author: !--author-handle-->@EFAcar<!--end-author-handle-- (Elif Acar) Repository: https://github.com/mlysy/LocalCop Branch with paper.md (empty if default branch): joss Version: v0.0.2 Editor: !--editor-->@lrnv<!--end-editor-- Reviewers: @AlexisDerumigny, @mingzehuang Archive: 10.5281/zenodo.13801203
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
@AlexisDerumigny & @mingzehuang, 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 @lrnv 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 @mingzehuang
π Checklist for @AlexisDerumigny