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

[PRE REVIEW]: The Python Sky Model 3 software #3534

Closed
whedon opened this issue Jul 24, 2021 · 33 comments
Closed

[PRE REVIEW]: The Python Sky Model 3 software #3534

whedon opened this issue Jul 24, 2021 · 33 comments

Comments

@whedon
Copy link

whedon commented Jul 24, 2021

Submitting author: @zonca (Andrea Zonca)
Repository: https://github.com/galsci/pysm
Version: 3.3.1
Editor: @christinahedges
Reviewers: @patricialarsen, @smsharma
Managing EiC: Daniel S. Katz

⚠️ JOSS reduced service mode ⚠️

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

Status

status

Status badge code:

HTML: <a href="https://joss.theoj.org/papers/8f2d6c3bbf6cbeffbb403a1207fa8de7"><img src="https://joss.theoj.org/papers/8f2d6c3bbf6cbeffbb403a1207fa8de7/status.svg"></a>
Markdown: [![status](https://joss.theoj.org/papers/8f2d6c3bbf6cbeffbb403a1207fa8de7/status.svg)](https://joss.theoj.org/papers/8f2d6c3bbf6cbeffbb403a1207fa8de7)

Author instructions

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

@zonca 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
Copy link
Author

whedon commented Jul 24, 2021

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

⚠️ JOSS reduced service mode ⚠️

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

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
Copy link
Author

whedon commented Jul 24, 2021

Wordcount for paper.md is 1150

@whedon
Copy link
Author

whedon commented Jul 24, 2021

Software report (experimental):

github.com/AlDanial/cloc v 1.88  T=0.11 s (639.8 files/s, 103335.1 lines/s)
--------------------------------------------------------------------------------
Language                      files          blank        comment           code
--------------------------------------------------------------------------------
Python                           42            820           1400           2965
Jupyter Notebook                  8              0           4416            527
reStructuredText                  9            152             26            313
TeX                               1              8              0            186
DOS Batch                         1             21              1            150
make                              1             22              5            106
INI                               1             14              0             90
YAML                              3             12             22             85
Markdown                          1             22              0             74
Bourne Shell                      2              0              0             12
Bourne Again Shell                1              1              5              5
TOML                              1              2              0              5
--------------------------------------------------------------------------------
SUM:                             71           1074           5875           4518
--------------------------------------------------------------------------------


Statistical information for the repository '47b8990881cbbcdb2803be82' was
gathered on 2021/07/24.
The following historical commit information, by author, was found:

Author                     Commits    Insertions      Deletions    % of changes
Andrea Zonca                   218          6873           3271           79.11
Ben Thorne                      21           706            173            6.86
NicolettaK                       1            21             15            0.28
Xavier Garrido                   3            51             27            0.61
ben thorne                      11          1345            340           13.14

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
Andrea Zonca               4696           68.3         23.0               12.18
Ben Thorne                  102           14.4         29.2               14.71
NicolettaK                   17           81.0         11.5                0.00
Xavier Garrido               41           80.4         16.1                4.88
ben thorne                  329           24.5         25.7               15.20

@whedon
Copy link
Author

whedon commented Jul 24, 2021

Failed to discover a valid open source license.

@whedon
Copy link
Author

whedon commented Jul 24, 2021

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1086/427976 is OK
- 10.21105/joss.01298 is OK
- 10.1093/mnras/stx949 is OK
- 10.1088/0067-0049/208/2/20 is OK
- 10.1145/2833157.2833162 is OK
- 10.1051/0004-6361/201321494 is OK
- 10.1051/0004-6361/201322068 is OK
- 10.3847/1538-3881/aabc4f is OK

MISSING DOIs

- None

INVALID DOIs

- None

@whedon
Copy link
Author

whedon commented Jul 24, 2021

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

@danielskatz
Copy link

@zonca - thanks for your submission.
Due to a current shortage of editors (that we are working to resolve), I'm going to put it on our waiting list until an appropriate editor is available.

@danielskatz danielskatz added the waitlisted Submissions in the JOSS backlog due to reduced service mode. label Jul 25, 2021
@zonca
Copy link

zonca commented Jul 30, 2021

thanks @danielskatz
Can I post a pre-print of the paper to the Arxiv in the meantime?

@danielskatz
Copy link

Yes, that's fine. Also, be aware that you need to have an open source license - I see what I think is a BSD 3-clause license in a subdirectory, which JOSS did not find, so this is technically ok. Normally, this would be in the top level of the repo and would be called LICENSE so that GitHub knows it is there and labels the repo correctly and JOSS knows it's there. This placement is not required by JOSS, but is a common practice.

@zonca
Copy link

zonca commented Aug 4, 2021

@whedon commands

@whedon
Copy link
Author

whedon commented Aug 4, 2021

Here are some things you can ask me to do:

# List Whedon's capabilities
@whedon commands

# List of editor GitHub usernames
@whedon list editors

# List of reviewers together with programming language preferences and domain expertise
@whedon list reviewers

EDITORIAL TASKS

# Compile the paper
@whedon generate pdf

# Compile the paper from alternative branch
@whedon generate pdf from branch custom-branch-name

# Ask Whedon to check the references for missing DOIs
@whedon check references

# Ask Whedon to check repository statistics for the submitted software
@whedon check repository

@zonca
Copy link

zonca commented Aug 4, 2021

thanks @danielskatz, I moved the license to the root of the repository, I don't know how to tell whedon to check again.

I also published the preprint to Arxiv: https://arxiv.org/abs/2108.01444

@danielskatz
Copy link

Thanks - hopefully a suitable editor will become available soon.

@arfon
Copy link
Member

arfon commented Aug 15, 2021

@whedon assign me as editor

@whedon
Copy link
Author

whedon commented Aug 15, 2021

OK, the editor is @arfon

@arfon arfon removed the waitlisted Submissions in the JOSS backlog due to reduced service mode. label Aug 15, 2021
@arfon
Copy link
Member

arfon commented Aug 15, 2021

👋 @zonca – many thanks for your submission to JOSS! Could you take a look a this list of potential reviewers and identify a few people who would be good candidates to review this submission?

@zonca
Copy link

zonca commented Aug 16, 2021

thanks @arfon!

first choice would be cosmology + python, specific CMB knowledge first: Pqrs6, sibirrer, smsharma, benjaminrose, danehkar, yymao, aureliocarnero, changhoonhahn, Christovis, rmorgan10, villaa

@arfon
Copy link
Member

arfon commented Aug 26, 2021

@zonca - is this submission associated with a paper in a AAS journal?

@arfon
Copy link
Member

arfon commented Aug 26, 2021

👋 @sibirrer @yymao @changhoonhahn – would any of you be willing to review this submission for JOSS? We carry out our checklist-driven reviews here in GitHub issues and follow these guidelines: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html

The submission under consideration here is The Python Sky Model 3 software.

@zonca
Copy link

zonca commented Aug 26, 2021

@zonca - is this submission associated with a paper in a AAS journal?

No

@zonca
Copy link

zonca commented Sep 17, 2021

@arfon no answers, do you plan to ask someone else? Thanks!

@sibirrer
Copy link

sorry for not having responded @arfon ! I am currently overwhelmed with other tasks and have to pass on this (very nice) software review.

@arfon
Copy link
Member

arfon commented Sep 19, 2021

👋 @aureliocarnero – would you be willing to review this submission for JOSS? We carry out our checklist-driven reviews here in GitHub issues and follow these guidelines: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html

The submission under consideration here is The Python Sky Model 3 software.

@arfon
Copy link
Member

arfon commented Sep 28, 2021

@whedon assign @christinahedges as editor

@whedon whedon assigned christinahedges and unassigned arfon Sep 28, 2021
@whedon
Copy link
Author

whedon commented Sep 28, 2021

OK, the editor is @christinahedges

@arfon
Copy link
Member

arfon commented Sep 28, 2021

👋 @zonca@christinahedges has kindly agreed to take on editing this submission. Thanks @christinahedges

@christinahedges
Copy link

👋 @zonca - thanks for your submission and patience! I'm going to be your new editor now and we'll find some reviewers for your submission and get your review started ASAP.

@christinahedges
Copy link

@whedon assign @patricialarsen as reviewer

@whedon
Copy link
Author

whedon commented Sep 28, 2021

OK, @patricialarsen is now a reviewer

@christinahedges
Copy link

@whedon add @smsharma as reviewer

@whedon
Copy link
Author

whedon commented Sep 29, 2021

OK, @smsharma is now a reviewer

@christinahedges
Copy link

@whedon start review

@whedon
Copy link
Author

whedon commented Sep 29, 2021

OK, I've started the review over in #3783.

@whedon whedon closed this as completed Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants