Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign up[REVIEW]: SunPy: A Python package for Solar Physics #1832
Comments
This comment has been minimized.
This comment has been minimized.
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @mwcraig, @ceb8 it looks like you're currently assigned to review this paper If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews To fix this do the following two things:
For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf from branch joss_paper |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1832 with the following error: ORCID looks to be the wrong length |
This comment has been minimized.
This comment has been minimized.
@Cadair - could you fix that ORCID please |
This comment has been minimized.
This comment has been minimized.
@mwcraig, @ceb8 - one thing I forgot to mention when inviting you both to review is that this JOSS submission/paper is a joint publication with AAS Journals, i.e. this JOSS paper will be published together with another paper in one of the AAS journals. As part of this collaboration, AAS publishing makes a small donation to the running costs of JOSS. If this is unacceptable to you, please let me know and I can look for alternative reviewers. Some related links:
|
This comment has been minimized.
This comment has been minimized.
The author list is currently not in the right format. I am still soliciting ORCIDs and Affiliations from people, I will reformat the author list and create a zenodo author file at some point soon. I can do this at whatever point you feel is most appropriate, but am stalling to give people time to give me their info. |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf from branch joss_paper |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1832 with the following error: /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-1e4ee47b240d/lib/whedon/author.rb:56:in |
This comment has been minimized.
This comment has been minimized.
I fixed the ORCID, but that error is because the author list is just in the wrong format. |
This comment has been minimized.
This comment has been minimized.
@mwcraig, @ceb8 - for now, please use this copy of the paper for your review for now: https://github.com/openjournals/joss-papers/blob/joss.01789/joss.01789/10.21105.joss.01789.pdf |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@arfon -- sorry for the delay. I'll get the review done on Monday. |
This comment has been minimized.
This comment has been minimized.
@arfon -- this link is a 404... |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1832 with the following error: Can't find any papers to compile :-( |
This comment has been minimized.
This comment has been minimized.
@mwcraig - this paper should work for now: 10.21105.joss.01832.pdf |
This comment has been minimized.
This comment has been minimized.
@Cadair - please fix up your paper authors and affiliations ASAP so that the reviewers can check it out. As it currently stands, the paper won't compile. |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf from branch joss_paper |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Well I made it compile by doing that particular hack. My author list is being generated by a yaml translation script now, so it's easy enough to edit |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf from branch joss_paper |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
You can also have an affiliation of 'None' for some of the people (where |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf from branch joss_paper |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
Done |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@arfon -- I made a comment on the sunpy JOSS paper that they hadn't addressed the state of the field in the JOSS paper. @mbobra mentioned that it is addressed in the accompanying ApJ paper:
I've read that section of the ApJ paper and it is very good and probably too long for the JOSS paper. Does it need to be addressed in the JOSS paper? Would a reference to the ApJ paper suffice? My leaning would be towards a reference in the JOSS paper to the discussion in the ApJ paper but I haven't reviewed a JOSS/ApJ combination before. |
This comment has been minimized.
This comment has been minimized.
I think this is OK, we just need to make sure it's cited properly in the body of the text. |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf from branch joss_paper |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Thanks for the review @mwcraig |
This comment has been minimized.
This comment has been minimized.
Aside from the incomplete ApJ paper reference issue noted by @mwcraig this is ready. |
This comment has been minimized.
This comment has been minimized.
@Cadair - I don't think we can finalize the ApJ citation until we have a DOI for this paper. @crawfordsm - do we have a DOI yet for the accompanying ApJ paper? |
This comment has been minimized.
This comment has been minimized.
Thanks @ceb8 |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf from branch joss_paper |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@arfon We would like to document how we have sorted our author list somewhere, in the name of transparency and clarity. Do you have any suggestions about where we should put that information? |
This comment has been minimized.
This comment has been minimized.
Something like this will allow you to have a footnote in the paper: sunpy/sunpy@joss_paper...arfon:patch-1 |
This comment has been minimized.
This comment has been minimized.
|
whedon commentedOct 23, 2019
•
edited by ceb8
Submitting author: @Cadair (Stuart Mumford)
Repository: https://github.com/sunpy/sunpy
Version: v1.0.0
Editor: @arfon
Reviewer: @mwcraig, @ceb8
Archive: Pending
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
@mwcraig & @ceb8, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @arfon know.
Review checklist for @mwcraig
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @ceb8
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper