Skip to content
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

Software review #7

Open
9 of 42 tasks
xuanxu opened this issue Oct 21, 2020 · 57 comments
Open
9 of 42 tasks

Software review #7

xuanxu opened this issue Oct 21, 2020 · 57 comments
Labels

Comments

@xuanxu
Copy link
Member

xuanxu commented Oct 21, 2020

Submitting Author: JJ (@xuanxu)
Repository: https://github.com/xuanxu/intergalactic
Version submitted: 1.3

Editor: TBD
Reviewers: Alice, Kate
Archive: TBD
Version accepted: TBD


Description

This software is a Q-Matrices generator.

Based on explicit values for solar abundances, z and IMF, it calculates matrices Q(i,j) of masses of 
elements i ejected to the galactic medium as element j, for a complete range of stellar masses, 
accounting for supernovas of types Ia and II. 
You can read more about the Matrices Q formalism in Ferrini et al. 1992.

Intergalactic computes the contribution matrix of 15 elements:

H  D  He3  He4  C  C13  N  O  n.r.  Ne  Mg  Si  S  Ca  Fe

Scope

  • Please indicate which category or categories this package falls under: (Please check an appropriate box below. If you are unsure, we suggest you make a pre-submission inquiry.):

    • data retrieval
    • scientific software wrappers
    • field and lab reproducibility tools
    • geospatial data
    • text analysis

Technical checks

Confirm each of the following by checking the box.
This package:

  • does not violate the Terms of Service of any service it interacts with.
  • has a CRAN and OSI accepted license.
  • contains a README with instructions for installing the development version.
  • includes documentation.
  • has a test suite.
  • has continuous integration.

Code of conduct

  • I agree to abide by our Code of Conduct during the review process and in maintaining my package should it be accepted.

Review checklist for @xuanxu

Conflict of interest

  • I confirm that I have no COIs with reviewing this work or that any perceived COIs have been waived by an editor for the purpose of this review.

Code of Conduct

  • I confirm that I read and will adhere to the code of conduct.

General checks

  • Repository: Is the source code for this software available at the repo url?
  • License: Does the repository contain a plain-text LICENSE file with the contents of an OSI approved software license?
  • Contribution and authorship: Has the submitting author (@xuanxu) made major contributions to the software?
  • Substantial scholarly effort: Does this submission meet the scope eligibility?

Functionality

  • Installation: Does installation proceed as outlined in the documentation?
  • Functionality: Have the functional claims of the software been confirmed?
  • Performance: If there are any performance claims of the software, have they been confirmed? (If there are no claims, please check off this item.)

Documentation

  • A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
  • Installation instructions: Is there a clearly-stated list of dependencies? Ideally these should be handled with an automated package management solution.
  • Example usage: Do the authors include examples of how to use the software (ideally to solve real-world analysis problems).
  • Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)?
  • Automated tests: Are there automated tests or manual steps described so that the functionality of the software can be verified?
  • Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support

Review checklist for @gemgon

Conflict of interest

  • I confirm that I have no COIs with reviewing this work or that any perceived COIs have been waived by an editor for the purpose of this review.

Code of Conduct

  • I confirm that I read and will adhere to the code of conduct.

General checks

  • Repository: Is the source code for this software available at the repo url?
  • License: Does the repository contain a plain-text LICENSE file with the contents of an OSI approved software license?
  • Contribution and authorship: Has the submitting author (@xuanxu) made major contributions to the software?
  • Substantial scholarly effort: Does this submission meet the scope eligibility?

Functionality

  • Installation: Does installation proceed as outlined in the documentation?
  • Functionality: Have the functional claims of the software been confirmed?
  • Performance: If there are any performance claims of the software, have they been confirmed? (If there are no claims, please check off this item.)

Documentation

  • A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
  • Installation instructions: Is there a clearly-stated list of dependencies? Ideally these should be handled with an automated package management solution.
  • Example usage: Do the authors include examples of how to use the software (ideally to solve real-world analysis problems).
  • Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)?
  • Automated tests: Are there automated tests or manual steps described so that the functionality of the software can be verified?
  • Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support
@botsci
Copy link
Member

botsci commented Oct 21, 2020

Welcome xuanxu, thanks for opening this issue 🎉

Thanks for sending your software to aliadalabs/sunnydale

@xuanxu
Copy link
Member Author

xuanxu commented Oct 21, 2020

@botsci help

@botsci
Copy link
Member

botsci commented Oct 21, 2020

Hello @xuanxu, here are the things you can ask me to do:


# List all available commands
@botsci help

# Assign a user as the reviewer N of this submission (where N=1,2...)
@botsci assign @username as reviewer 2

# Remove the user assigned as reviewer N of this submission (where N=1,2...)
@botsci remove reviewer 2

# Assign a user as the editor of this submission
@botsci assign @username as editor

# Remove the editor assigned to this submission
@botsci remove editor

# Send an invitation to a user to collaborate in the review
@botsci invite @username

# Add a user to this issue's assignees list
@botsci add assignee: @username

# Remove a user from this issue's assignees list
@botsci remove assignee: @username

# Set a value for version
@botsci set v1.0.0 as version

# Set a value for archive
@botsci set 10.21105/joss.12345 as archive

# Label issue with: reviewed, recommend publication, pending publication. Remove labels: ongoing, pending review
@botsci review successful

# Label issue with: recommend rejection
@botsci review failed

# Label the issue with: [rejected] and close it.
@botsci reject

# Reports on the quality of the code
@botsci analyze code

# Random facts about cats
@botsci tell me something about cats

# Add review checklist for a user
@botsci add checklist for @username

# Remove the checklist for a user
@botsci remove checklist for @username

@xuanxu
Copy link
Member Author

xuanxu commented Oct 21, 2020

@botsci remove checklist for @gemgon

@botsci
Copy link
Member

botsci commented Oct 21, 2020

There is not a checklist for @gemgon

@xuanxu
Copy link
Member Author

xuanxu commented Oct 21, 2020

@botsci add checklist for @gemgon

@botsci
Copy link
Member

botsci commented Oct 21, 2020

Checklist added for @gemgon

@xuanxu
Copy link
Member Author

xuanxu commented Oct 21, 2020

@botsci add checklist for @xuanxu

@botsci
Copy link
Member

botsci commented Oct 21, 2020

Checklist added for @xuanxu

@xuanxu
Copy link
Member Author

xuanxu commented Oct 21, 2020

@botsci remove checklist for @gemgon

@botsci
Copy link
Member

botsci commented Oct 21, 2020

There is not a checklist for @gemgon

@xuanxu
Copy link
Member Author

xuanxu commented Oct 21, 2020

@botsci remove checklist for @gemgon

@botsci
Copy link
Member

botsci commented Oct 21, 2020

Checklist for @gemgon removed

@xuanxu
Copy link
Member Author

xuanxu commented Oct 21, 2020

@botsci add checklist for @gemgon

@botsci
Copy link
Member

botsci commented Oct 21, 2020

Checklist added for @gemgon

@xuanxu
Copy link
Member Author

xuanxu commented Nov 3, 2020

@botsci help

@botsci
Copy link
Member

botsci commented Nov 3, 2020

Hello @xuanxu, here are the things you can ask me to do:


# List all available commands
@botsci help

# Assign a user as the reviewer N of this submission (where N=1,2...)
@botsci assign @username as reviewer 2

# Remove the user assigned as reviewer N of this submission (where N=1,2...)
@botsci remove reviewer 2

# Assign a user as the editor of this submission
@botsci assign @username as editor

# Remove the editor assigned to this submission
@botsci remove editor

# Send an invitation to a user to collaborate in the review
@botsci invite @username

# Add a user to this issue's assignees list
@botsci add assignee: @username

# Remove a user from this issue's assignees list
@botsci remove assignee: @username

# Set a value for version
@botsci set v1.0.0 as version

# Set a value for archive
@botsci set 10.21105/joss.12345 as archive

# Label issue with: reviewed, recommend publication, pending publication. Remove labels: ongoing, pending review
@botsci review successful

# Label issue with: recommend rejection
@botsci review failed

# Label the issue with: [rejected] and close it.
@botsci reject

# Reports on the quality of the code
@botsci analyze code

# Random facts about cats
@botsci tell me something about cats

# Add review checklist for a user
@botsci add checklist for @username

# Remove the checklist for a user
@botsci remove checklist for @username

@xuanxu
Copy link
Member Author

xuanxu commented Nov 7, 2020

@botsci help

@botsci
Copy link
Member

botsci commented Nov 7, 2020

Hello @xuanxu, here are the things you can ask me to do:


# List all available commands
@botsci help

# Assign a user as the reviewer N of this submission (where N=1,2...)
@botsci assign @username as reviewer 2

# Remove the user assigned as reviewer N of this submission (where N=1,2...)
@botsci remove reviewer 2

# Assign a user as the editor of this submission
@botsci assign @username as editor

# Remove the editor assigned to this submission
@botsci remove editor

# Send an invitation to a user to collaborate in the review
@botsci invite @username

# Add a user to this issue's assignees list
@botsci add assignee: @username

# Remove a user from this issue's assignees list
@botsci remove assignee: @username

# Check the references of the paper for missing DOIs
# Optionally, it can be run on a non-default branch 
@botsci check references
@botsci check references from custom-branch-name

# Set a value for version
@botsci set v1.0.0 as version

# Set a value for archive
@botsci set 10.21105/joss.12345 as archive

# Label issue with: reviewed, recommend publication, pending publication. Remove labels: ongoing, pending review
@botsci review successful

# Label issue with: recommend rejection
@botsci review failed

# Label the issue with: [rejected] and close it.
@botsci reject

# Reports on the quality of the code
@botsci analyze code

# Random facts about cats
@botsci tell me something about cats

# Add review checklist for a user
@botsci add checklist for @username

# Remove the checklist for a user
@botsci remove checklist for @username

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci add juanjo to reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

I can't find the reviewers list

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci add juanjo from reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

That command is confusing. Did you mean to ADD TO reviewers or to REMOVE FROM reviewers?

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci add juanjo to reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

juanjo added to the reviewers list!

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci remove juanjo from reviewers

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci remove laura from reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

laura is not in the reviewers list

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci remove juanjo from reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

juanjo removed from the reviewers list!

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci add John to reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

John added to the reviewers list!

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci add Kate to reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

Kate added to the reviewers list!

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci add Alice to reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

Alice added to the reviewers list!

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci remove Kate from reviewers

@botsci
Copy link
Member

botsci commented Nov 20, 2020

Kate removed from the reviewers list!

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

thanks @botsci

@botsci
Copy link
Member

botsci commented Nov 20, 2020

You are welcome

@xuanxu
Copy link
Member Author

xuanxu commented Nov 20, 2020

@botsci check repository

@botsci
Copy link
Member

botsci commented Nov 20, 2020

Software report:

github.com/AlDanial/cloc v 1.88  T=0.09 s (391.6 files/s, 37195.4 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          22            619            304           1592
reStructuredText                 8            172             40            361
Markdown                         2             37              0             95
YAML                             2             12             42             31
make                             1              4              6              9
-------------------------------------------------------------------------------
SUM:                            35            844            392           2088
-------------------------------------------------------------------------------


Statistical information for the repository 'a78ccc90f72ffa12721304ce' was
gathered on 2020/11/20.
The following historical commit information, by author, was found:

Author                     Commits    Insertions      Deletions    % of changes
Juanjo Bazán                    8           547             33            7.24
Juanjo Bazán                   352          5267           2167           92.76

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
Juanjo Bazán               3374           64.1         15.9                5.63

@botsci botsci added the Python label Nov 20, 2020
@botsci
Copy link
Member

botsci commented Nov 20, 2020

Failed to discover a Statement of need section in paper

@xuanxu
Copy link
Member Author

xuanxu commented Nov 23, 2020

@botsci add Kate to reviewers

@botsci
Copy link
Member

botsci commented Nov 23, 2020

Kate added to the reviewers list!

@xuanxu
Copy link
Member Author

xuanxu commented Nov 23, 2020

@botsci remove John from reviewers

@botsci
Copy link
Member

botsci commented Nov 23, 2020

John removed from the reviewers list!

@xuanxu
Copy link
Member Author

xuanxu commented Nov 26, 2020

@botsci code of conduct

@botsci
Copy link
Member

botsci commented Nov 26, 2020

@botsci
Copy link
Member

botsci commented Nov 26, 2020

It's adapted from the Contributor Covenant: http://contributor-covenant.org

@botsci
Copy link
Member

botsci commented Nov 26, 2020

Reports of abusive or harassing behavior may be reported to [email protected]

@xuanxu
Copy link
Member Author

xuanxu commented Nov 26, 2020

@botsci list editors

@botsci
Copy link
Member

botsci commented Nov 26, 2020

Here is the list of current topic editors:

Astronomy

- astroeditor_33
- astro_editor_2

Social science

- sociology_editor_1
- sociology_editor_2

Genomics

- @xuanxu
- genomics_editor_7

Machine Learning

- Alice
- Bob

Thanks for asking, xuanxu.

@xuanxu
Copy link
Member Author

xuanxu commented Nov 26, 2020

@botsci help

@botsci
Copy link
Member

botsci commented Nov 26, 2020

Hello @xuanxu, here are the things you can ask me to do:


# List all available commands
@botsci help

# Assign a user as the reviewer N of this submission (where N=1,2...)
@botsci assign @username as reviewer 2

# Remove the user assigned as reviewer N of this submission (where N=1,2...)
@botsci remove reviewer 2

# Add to this issue's reviewers list
@botsci add @username to reviewers

# Remove from this issue's reviewers list
@botsci remove @username from reviewers

# Assign a user as the editor of this submission
@botsci assign @username as editor

# Remove the editor assigned to this submission
@botsci remove editor

# Send an invitation to a user to collaborate in the review
@botsci invite @username

# Add a user to this issue's assignees list
@botsci add assignee: @username

# Remove a user from this issue's assignees list
@botsci remove assignee: @username

# Check the references of the paper for missing DOIs
# Optionally, it can be run on a non-default branch 
@botsci check references
@botsci check references from custom-branch-name

# Perform checks on the repository
# Optionally, it can be run on a non-default branch 
@botsci check repository
@botsci check repository from custom-branch-name

# Set a value for version
@botsci set v1.0.0 as version

# Set a value for archive
@botsci set 10.21105/joss.12345 as archive

# Label issue with: reviewed, recommend publication, pending publication. Remove labels: ongoing, pending review
@botsci review successful

# Label issue with: recommend rejection
@botsci review failed

# Close the issue
@botsci reject

# Reports on the quality of the code
@botsci analyze code

# Random facts about cats
@botsci tell me something about cats

# Add review checklist for a user
@botsci add checklist for @username

# Remove the checklist for a user
@botsci remove checklist for @username

@xuanxu
Copy link
Member Author

xuanxu commented Nov 26, 2020

@botsci help

@botsci
Copy link
Member

botsci commented Nov 26, 2020

Hello @xuanxu, here are the things you can ask me to do:


# List all available commands
@botsci help

# Replies to code of conduct
@botsci code of conduct

# Replies to list editors
@botsci list editors

# Assign a user as the reviewer N of this submission (where N=1,2...)
@botsci assign @username as reviewer 2

# Remove the user assigned as reviewer N of this submission (where N=1,2...)
@botsci remove reviewer 2

# Add to this issue's reviewers list
@botsci add @username to reviewers

# Remove from this issue's reviewers list
@botsci remove @username from reviewers

# Assign a user as the editor of this submission
@botsci assign @username as editor

# Remove the editor assigned to this submission
@botsci remove editor

# Send an invitation to a user to collaborate in the review
@botsci invite @username

# Add a user to this issue's assignees list
@botsci add assignee: @username

# Remove a user from this issue's assignees list
@botsci remove assignee: @username

# Check the references of the paper for missing DOIs
# Optionally, it can be run on a non-default branch 
@botsci check references
@botsci check references from custom-branch-name

# Perform checks on the repository
# Optionally, it can be run on a non-default branch 
@botsci check repository
@botsci check repository from custom-branch-name

# Set a value for version
@botsci set v1.0.0 as version

# Set a value for archive
@botsci set 10.21105/joss.12345 as archive

# Label issue with: reviewed, recommend publication, pending publication. Remove labels: ongoing, pending review
@botsci review successful

# Label issue with: recommend rejection
@botsci review failed

# Close the issue
@botsci reject

# Reports on the quality of the code
@botsci analyze code

# Random facts about cats
@botsci tell me something about cats

# Add review checklist for a user
@botsci add checklist for @username

# Remove the checklist for a user
@botsci remove checklist for @username

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants