openjournals / joss-reviews

Reviews for the Journal of Open Source Software
Creative Commons Zero v1.0 Universal
707 stars 37 forks source link

[PRE REVIEW]: CarrierCapture.jl: Anharmonic Carrier Capture #2091

Closed whedon closed 4 years ago

whedon commented 4 years ago

Submitting author: @frssp (Sunghyun Kim) Repository: https://github.com/WMD-group/CarrierCapture.jl Version: v0.1 Editor: @danielskatz Reviewers: @utf, @taylor-a-barnes Managing EiC: Daniel S. Katz

Author instructions

Thanks for submitting your paper to JOSS @frssp. Currently, there isn't an JOSS editor assigned to your paper.

@frssp if you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). In addition, this list of people have already agreed to review for JOSS and may be suitable for this submission (please start at the bottom of the list).

Editor instructions

The JOSS submission bot @whedon is here to help you find and assign reviewers and start the main review. To find out what @whedon can do for you type:

@whedon commands
whedon commented 4 years ago

Hello human, I'm @whedon, a robot that can help you with some common editorial tasks.

For a list of things I can do to help you, just type:

@whedon commands

For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:

@whedon generate pdf
whedon commented 4 years ago
Software report (experimental):

github.com/AlDanial/cloc v 1.84  T=0.09 s (441.2 files/s, 54128.0 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Julia                           16            284            112           1112
Markdown                         9            197              0            584
Python                           4             92             50            236
Jupyter Notebook                 4              0           1818            224
YAML                             5             15             29            179
TeX                              1              7              0             63
TOML                             2              1              0             27
-------------------------------------------------------------------------------
SUM:                            41            596           2009           2425
-------------------------------------------------------------------------------

Statistical information for the repository '2091' was gathered on 2020/02/10.
The following historical commit information, by author, was found:

Author                     Commits    Insertions      Deletions    % of changes
Sunghyun Kim                     4           101            234           35.19
frssp                            3           563             54           64.81

Below are the number of rows from each author that have survived and are still
intact in the current revision:

Author                     Rows      Stability          Age       % in comments
Sungyun Kim                 378          100.0          9.6                8.20
whedon commented 4 years ago
Reference check summary:

OK DOIs

- None

MISSING DOIs

- https://doi.org/10.1088/0034-4885/44/12/001 may be missing for title: Non-radiative transitions in semiconductors
- https://doi.org/10.1103/physrevb.90.075202 may be missing for title: First-principles theory of nonradiative carrier capture via multiphonon emission
- https://doi.org/10.1103/physrevb.100.041202 may be missing for title: Anharmonic lattice relaxation during nonradiative carrier capture

INVALID DOIs

- None
whedon commented 4 years ago

:point_right: Check article proof :page_facing_up: :point_left:

danielskatz commented 4 years ago

šŸ‘‹ @frssp - there seem to be some DOIs missing in your bib file - please add any that are correct (noting that whedon's reference check summary sometimes makes mistakes)

frssp commented 4 years ago

Updated. @whedon generate pdf

whedon commented 4 years ago

PDF failed to compile for issue #2091 with the following error:

Error reading bibliography ./paper.bib (line 14, column 3): unexpected "d" expecting space, ",", white space or "}" Error running filter pandoc-citeproc: Filter returned error status 1 Looks like we failed to compile the PDF

frssp commented 4 years ago

@whedon generate pdf

whedon commented 4 years ago

:point_right: Check article proof :page_facing_up: :point_left:

danielskatz commented 4 years ago

Thanks - one of the DOIs has a leading "https:/doi.org/" that should be removed

danielskatz commented 4 years ago

@whedon assign @danielskatz as editor

whedon commented 4 years ago

OK, the editor is @danielskatz

danielskatz commented 4 years ago

If you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). In addition, this list of people have already agreed to review for JOSS and may be suitable for this submission (please start at the bottom of the list).

frssp commented 4 years ago

I suggest JMSkelton and utf for potential reviewers because they are experts in computational chemistry.

frssp commented 4 years ago

DOI fixed; @whedon generate pdf

danielskatz commented 4 years ago

@whedon generate pdf has to be the start (or entirety) of a comment

danielskatz commented 4 years ago

@whedon generate pdf

whedon commented 4 years ago

:point_right: Check article proof :page_facing_up: :point_left:

danielskatz commented 4 years ago

šŸ‘‹ @JMSkelton & @utf - would either or both of you be willing to review this submission?

utf commented 4 years ago

Iā€™d be happy to review this.

danielskatz commented 4 years ago

šŸ‘‹ @godotalgorithm & @taylor-a-barnes - would either or both of you be willing to review this submission to JOSS? JOSS reviews are of the software and short paper, and are checklist-driven, with the goal of improving the software to the point where it can be accepted - see https://joss.readthedocs.io/en/latest/reviewer_guidelines.html for more info

danielskatz commented 4 years ago

Thanks @utf - I'll assign you, but we won't start until we get at least one more reviewer.

danielskatz commented 4 years ago

@whedon assign @utf as reviewer

whedon commented 4 years ago

OK, @utf is now a reviewer

taylor-a-barnes commented 4 years ago

@danielskatz I would be happy to review this submission.

danielskatz commented 4 years ago

Thanks @taylor-a-barnes - I'll add you, and we'll start the review process in a new thread - hang on...

danielskatz commented 4 years ago

@whedon add @taylor-a-barnes as reviewer

whedon commented 4 years ago

OK, @taylor-a-barnes is now a reviewer

danielskatz commented 4 years ago

@whedon start review

whedon commented 4 years ago

OK, I've started the review over in https://github.com/openjournals/joss-reviews/issues/2102.