Closed editorialbot closed 4 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.08 s (976.6 files/s, 295827.1 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
R 32 980 2907 4283
C++ 17 809 2297 4256
C/C++ Header 16 487 2160 1611
TeX 2 120 0 891
Rmd 6 536 820 868
Markdown 5 185 0 695
YAML 1 3 4 18
-------------------------------------------------------------------------------
SUM: 79 3120 8188 12622
-------------------------------------------------------------------------------
gitinspector failed to run statistical information for the repository
Wordcount for paper.md
is 778
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.21105/joss.00860 is OK
- 10.1007/s00454-002-2885-2 is OK
- 10.1007/s00454-004-1146-y is OK
- 10.1523/eneuro.0543-19.2020 is OK
- 10.32614/rj-2011-002 is OK
- 10.1038/s41598-021-88027-8 is OK
- 10.18653/v1/2022.repl4nlp-1.18 is OK
- 10.1186/s13244-023-01413-w is OK
- 10.1007/978-3-540-33037-0_14 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:
@EduPH and @peekxc - Thanks for agreeing to review this submission.
This is the review thread for the paper. All of our communications will happen here from now on.
As mentioned above, you can use the command @editorialbot generate my checklist
to create your review checklist. As you go over the submission, please check any items that you feel have been satisfied.
There are also links to the JOSS reviewer guidelines (https://joss.readthedocs.io/en/latest/reviewer_guidelines.html)
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, reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention openjournals/joss-reviews#6321
so that a link is created to this thread for visibility. 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 you require additional time. We can also use editorialbot
(our bot) to set automatic reminders if you know you'll be away for a known period.
Please feel free to ping me (@mahfuz05062) if you have any questions/concerns.
Hi @peekxc , I just wanted to make sure that you have noticed this review thread. The instructions are in the first post, but please let me know if you have any questions. Thanks!
@peekxc and @EduPH I see that the reviews are moving along nicely! Please keep up the progress and let me know if there is anything I can do to help! Thanks!
I guess I can add it here. Related to the software paper references. It would be nice to add TDA manuals such as (at least one):
I guess I can add it here. Related to the software paper references. It would be nice to add TDA manuals such as (at least one):
- Computational Topology - an Introduction. H. Edelsbrunner, and J. Harer. American Mathematical Society, (2010)
- Dey TK, Wang Y. Computational Topology for Data Analysis. Cambridge University Press; 2022.
- Carlsson G, Vejdemo-Johansson M. Topological Data Analysis with Applications. Cambridge University Press; 2021.
@shaelebrown can you address this? Thanks!
Hi there! Sorry, I am submitting my thesis tomorrow and am very swamped. I've seen all the issues raised and plan on addressing them on Monday if that's alright :)
Hi there! Sorry, I am submitting my thesis tomorrow and am very swamped. I've seen all the issues raised and plan on addressing them on Monday if that's alright :)
No worries! Good luck with your thesis!
Hey guys I'm back (submitted on Friday :) ), thanks for your patience. @EduPH I added a reference for the TDA with applications manual in the introduction section of the TDApplied Theory and Practice vignette (the sentence starting with "For a broad...") as I am most familiar with that text.
@peekxc Thanks for completing the review! @EduPH is there anything else you are waiting for? Anything either me or the author can help with? Thanks!
@mahfuz05062 I've finished checking. Thank you!
@mahfuz05062 what are the next steps at this point?
@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
@shaelebrown please complete the author tasks listed above (if not already done).
@mahfuz05062 I confirm that:
I'll just need to update the paper.md file with those kinds of information, or does that get handled automatically?
@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 10.5281/zenodo.10814141 as archive
Done! archive is now 10.5281/zenodo.10814141
@editorialbot set v3.0.3 as version
Done! version is now v3.0.3
@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
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.21105/joss.00860 is OK
- 10.1007/s00454-002-2885-2 is OK
- 10.1007/s00454-004-1146-y is OK
- 10.1523/eneuro.0543-19.2020 is OK
- 10.32614/rj-2011-002 is OK
- 10.1038/s41598-021-88027-8 is OK
- 10.18653/v1/2022.repl4nlp-1.18 is OK
- 10.1186/s13244-023-01413-w is OK
- 10.1007/978-3-540-33037-0_14 is OK
MISSING DOIs
- No DOI given, and none found for title: TDA: Statistical Tools for Topological Data Analys...
- No DOI given, and none found for title: TDAstats: Pipeline for Topological Data Analysis
- No DOI given, and none found for title: devtools: Tools to Make Developing R Packages Easi...
- No DOI given, and none found for title: Hypothesis testing for topological data analysis
- No DOI given, and none found for title: Persistence Fisher Kernel: A Riemannian Manifold K...
INVALID DOIs
- None
@shaelebrown Can you check the missing DOI's above? Specially, 'Hypothesis testing for topological data analysis' and 'Persistence Fisher Kernel: A Riemannian ...' should have valid DOIs available as these are journal publications.
Hi @shaelebrown, did you get a chance to go through the DOI's mentioned above?
Hey @mahfuz05062 sorry I didn't see your message last week - just updated the two DOIs (on the JOSS branch) let me know if that looks better!
@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 references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.21105/joss.00860 is OK
- 10.1007/s00454-002-2885-2 is OK
- 10.1007/s00454-004-1146-y is OK
- 10.1007/s41468-017-0008-7 is OK
- 10.48550/arXiv.1802.03569 is OK
- 10.1523/eneuro.0543-19.2020 is OK
- 10.32614/rj-2011-002 is OK
- 10.1038/s41598-021-88027-8 is OK
- 10.18653/v1/2022.repl4nlp-1.18 is OK
- 10.1186/s13244-023-01413-w is OK
- 10.1007/978-3-540-33037-0_14 is OK
MISSING DOIs
- No DOI given, and none found for title: TDA: Statistical Tools for Topological Data Analys...
- No DOI given, and none found for title: TDAstats: Pipeline for Topological Data Analysis
- No DOI given, and none found for title: devtools: Tools to Make Developing R Packages Easi...
INVALID DOIs
- None
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
:wave: @openjournals/csism-eics, this paper is ready to be accepted and published.
Check final proof :point_right::page_facing_up: Download article
If the paper PDF and the deposit XML files look good in https://github.com/openjournals/joss-papers/pull/5175, then you can now move forward with accepting the submission by compiling again with the command @editorialbot accept
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.21105/joss.00860 is OK
- 10.1007/s00454-002-2885-2 is OK
- 10.1007/s00454-004-1146-y is OK
- 10.1007/s41468-017-0008-7 is OK
- 10.48550/arXiv.1802.03569 is OK
- 10.1523/eneuro.0543-19.2020 is OK
- 10.32614/rj-2011-002 is OK
- 10.1038/s41598-021-88027-8 is OK
- 10.18653/v1/2022.repl4nlp-1.18 is OK
- 10.1186/s13244-023-01413-w is OK
- 10.1007/978-3-540-33037-0_14 is OK
MISSING DOIs
- Errored finding suggestions for "TDA: Statistical Tools for Topological Data Analys...", please try later
- No DOI given, and none found for title: TDAstats: Pipeline for Topological Data Analysis
- No DOI given, and none found for title: devtools: Tools to Make Developing R Packages Easi...
INVALID DOIs
- None
All - Thanks for the work to this point.
As the AEiC for this track, I'll proofread this tomorrow and get back to @shaelebrown with next steps on their part, if any.
@shaelebrown - please change the title of your zenodo deposit to match the title of the paper. This is a change of metadata only, which doesn't require a new deposit or new DOI.
@shaelebrown - I've also suggested some small changes in https://github.com/shaelebrown/TDApplied/pull/5 - please merge this, or let me know what you disagree with, then we can proceed.
@danielskatz thanks for the notes - just updated the title to match on zenodo and merged your commits. Let me know if anything still needs fixing!
@editorialbot recommend-accept
Attempting dry run of processing paper acceptance...
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.21105/joss.00860 is OK
- 10.1007/s00454-002-2885-2 is OK
- 10.1007/s00454-004-1146-y is OK
- 10.1007/s41468-017-0008-7 is OK
- 10.48550/arXiv.1802.03569 is OK
- 10.1523/eneuro.0543-19.2020 is OK
- 10.32614/rj-2011-002 is OK
- 10.1038/s41598-021-88027-8 is OK
- 10.18653/v1/2022.repl4nlp-1.18 is OK
- 10.1186/s13244-023-01413-w is OK
- 10.1007/978-3-540-33037-0_14 is OK
MISSING DOIs
- No DOI given, and none found for title: TDA: Statistical Tools for Topological Data Analys...
- No DOI given, and none found for title: TDAstats: Pipeline for Topological Data Analysis
- No DOI given, and none found for title: devtools: Tools to Make Developing R Packages Easi...
INVALID DOIs
- None
Submitting author: !--author-handle-->@shaelebrown<!--end-author-handle-- (Shael Brown) Repository: https://github.com/shaelebrown/TDApplied Branch with paper.md (empty if default branch): JOSS Version: v3.0.3 Editor: !--editor-->@mahfuz05062<!--end-editor-- Reviewers: @EduPH, @peekxc Archive: 10.5281/zenodo.10814141
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
@EduPH & @peekxc, 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 @mahfuz05062 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 @EduPH
📝 Checklist for @peekxc