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]: OpenOA: An Open-Source Codebase For Operational Analysis of Wind Farms #2171
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. @gschivley 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.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
OK, @coroa is now a reviewer |
This comment has been minimized.
This comment has been minimized.
@sjpfenninger how strict is JOSS about requiring OSI approved licenses? It looks like the authors use a modified version of the BSD 3-clause. There are very minor changes to wording in the original 3 clauses, plus an extra clause:
|
This comment has been minimized.
This comment has been minimized.
We are very strict about this! We will not publish this submission with a modified license |
This comment has been minimized.
This comment has been minimized.
Thanks @danielskatz. Does that include minor modifications to the original clauses like those below?
NREL has other repos that use a standard BSD 3-Clause so I would think they should be able to change this one, I just want to be clear on exactly how strict JOSS is about deviations in wording. |
This comment has been minimized.
This comment has been minimized.
No deviations in wording are allowed. Any deviation is a different (and thus non-OSI-approved) license. |
This comment has been minimized.
This comment has been minimized.
@danielskatz @gschivley - I will bring this up with our team and see if we can accommodate this request. I imagine this license issue may have come up in the past for JOSS if they've published any work that was funded by the US government. |
This comment has been minimized.
This comment has been minimized.
In other cases that I remember, labs have found a way to use a standard license. |
This comment has been minimized.
This comment has been minimized.
@jordanperr - this is one example of another project making the change to a standard license (DoE/LBL funded work): #1370 |
whedon commentedMar 31, 2020
•
edited by gschivley
Submitting author: @jordanperr (Jordan Perr-Sauer)
Repository: https://github.com/NREL/OpenOA
Version: v1
Editor: @sjpfenninger
Reviewers: @gschivley, @coroa
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
@gschivley, 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 @sjpfenninger know.
Review checklist for @gschivley
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @coroa
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper