Repository navigation
Changes to JOSS paper based on editor comments - #127
Open
FlorianK13 wants to merge 12 commits into
Open
FlorianK13 wants to merge 12 commits into
FlorianK13 wants to merge 12 commits into
Conversation
FlorianK13
marked this pull request as draft
August 27, 2026 11:15
Member
Author
Not sure if this holds anymore, so better leave it out.
Also adding ANTS2d as new reference
FlorianK13
marked this pull request as ready for review
September 11, 2026 12:16
Member
Author
|
@AdamRJensen thanks again for the issues you raised and the work you are doing for the journal. Can you take a look at the changes and see if your issues are addressed properly? |
AdamRJensen
reviewed
Sep 28, 2026
AdamRJensen
reviewed
Sep 28, 2026
This was referenced Sep 28, 2026
AdamRJensen
reviewed
Sep 28, 2026
Contributor
|
Thanks for the good work @FlorianK13 |
Co-authored-by: Adam R. Jensen <39184289+AdamRJensen@users.noreply.github.com>
Member
Author
|
@AdamRJensen thanks for your recent comments - i added a new commit to address them. If you agree with the current changes, I would merge the PR? |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[JOSS] Consider citing pvlib ANTS2D model #118
[JOSS] Revise statement of need #119
[JOSS] pvlib should be noted as 2d #120
[JOSS] Does the shading mask correspond to cosine of angle of incidence? #121
[JOSS] How is GHI/DNI converted to radiances? #122
Software Design - Input datachapter. This is now a tradeoff: The paper got even longer, but the topic is not trivial.[JOSS] Is there a citation for the Munich cite-scale demonstration study? #123
Moreover, I went through the paper and fixed some more typos and wrong wording, and I also shortended it a bit. The paper is still longer than the usual JOSS paper - if needed I can once more look for text that can be reduced.