Add manuscript files for 2023 JOSS publication by kandersolar · Pull Request #1855 · pvlib/pvlib-python · GitHub
Skip to content

Add manuscript files for 2023 JOSS publication#1855

Merged
kandersolar merged 13 commits into
pvlib:mainfrom
kandersolar:joss2023
Sep 18, 2023
Merged

Add manuscript files for 2023 JOSS publication#1855
kandersolar merged 13 commits into
pvlib:mainfrom
kandersolar:joss2023

Conversation

@kandersolar

@kandersolar kandersolar commented Sep 13, 2023

Copy link
Copy Markdown
Member

@pvlib/pvlib-core here are draft files for the proposed 2023 JOSS publication. Please review and leave comments at your leisure.

I have archived the original files from the 2018 publication in a 2018 subdirectory. Unfortunately github is rendering the new files as diffs from the old rather than replacements. For convenience, I've configured a CI job to build a PDF which is a bit nicer to read. Here is the latest: paper.pdf. New versions should appear as artifacts within this PR's CI job.

Also, I've included the scripts that generate the figures so that they are in the PR's history for future reference, but plan to remove them before merging.

Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated

@AdamRJensen AdamRJensen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it might be nice to mention that the authors represent the core pvlib team.

Note, that I've provided a pre-review of the paper thus most of my feedback has been provided prior to this PR.

Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
kandersolar and others added 3 commits September 14, 2023 14:14
Co-authored-by: Adam R. Jensen <39184289+AdamRJensen@users.noreply.github.com>
Co-authored-by: Cliff Hansen <cwhanse@sandia.gov>
@kandersolar

Copy link
Copy Markdown
Member Author

@wholmgren wholmgren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a great paper - thanks! I didn't realize quite how much has changed since the initial JOSS paper but perhaps not surprising given that pvlib is now ~twice as old as it was then.

I think it's notable that we've doubled the number of maintainers, especially since it's the new ones that are driving this effort :)

Comment thread paper/paper.md Outdated
Comment thread paper/paper.md

@adriesse adriesse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely written!

I do think the author order merits some reflection. I'm always self-conscious of advocating for alphabetical given my name, but on the other hand it doesn't seem like such a bad option here. Of course there are huge differences in what we have contributed to this text and to pvlib overall, but we all operate under different constraints.

Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Comment thread paper/paper.md Outdated
Co-authored-by: Anton Driesse <anton.driesse@pvperformancelabs.com>
@kandersolar

kandersolar commented Sep 15, 2023

Copy link
Copy Markdown
Member Author

Thanks @wholmgren and @adriesse for the reviews. Here is the latest rendering: draft 2023-09-15.pdf

Regarding author order, I am -1 to alphabetical. If we do that here then it presumably sets a precedent for any future publications, and it's not fair for me to arbitrarily be the perpetual beneficiary of whatever significance people assign to the ordering.

I think the current ordering, other than myself, was based on github's reporting of commit counts here: https://github.com/pvlib/pvlib-python/graphs/contributors

If ordering by commits is a reasonable approach, perhaps it would make more sense to order by commits since the last paper: https://github.com/pvlib/pvlib-python/graphs/contributors?from=2018-09-07&to=2023-09-15&type=c

(edit: corrected the publication date of the paper in the above link)

Of course commit count only captures one dimension of contribution to the project, and rather loosely at that. Ideas for other objective ordering methods are welcome!

@AdamRJensen

AdamRJensen commented Sep 15, 2023

Copy link
Copy Markdown
Member

If ordering by commits is a reasonable approach, perhaps it would make more sense to order by commits since the last paper: https://github.com/pvlib/pvlib-python/graphs/contributors?from=2018-09-07&to=2023-09-15&type=c

I'm inclined to agree that this seems the best option, albeit not perfect.

@mikofski mikofski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great paper! I really like the milestone charts! And also the growth of functions! Mind blowing! It would be interesting in the section on criticality to list any PV technological breakthroughs as a result of research or models that leveraged pvlib — perhaps the next update? Regarding name order feel free to put my name last. Of course I’m proud to be associated with pvlib but I’m far from the most prolific contributor and I think Kevin and Adam are the most consistent voices I hear in the PR’s. Thanks for putting together such a cogent update!

Comment thread paper/paper.md
@kandersolar

Copy link
Copy Markdown
Member Author

Thanks everyone for the input and kind words. Since all of us have reviewed, I will merge this PR and begin the JOSS submission.

Note that we can still make additional manuscript changes if needed; JOSS reviews usually take a while to get going.

@kandersolar kandersolar merged commit 7fe065f into pvlib:main Sep 18, 2023
@kandersolar kandersolar deleted the joss2023 branch September 18, 2023 15:37
kandersolar added a commit to kandersolar/pvlib-python that referenced this pull request Sep 21, 2023
this file was accidentally merged in with pvlib#1855
kandersolar added a commit that referenced this pull request Sep 21, 2023
* 0.10.2 whatsnew cleanup

* create v0.10.3 whatsnew

* formatting tweaks

* remove GH actions workflow for joss-pdf

this file was accidentally merged in with #1855
@adriesse

Copy link
Copy Markdown
Member

I think ranking core team members is fundamentally undesirable. As I pointed out, we all operate under different constraints. This allows some of us to spend much more time on pvlib than others. It would probably be easier to change my name than to change my constraints. Furthermore, the core team is not homogeneous, so contributions will vary in nature and size and countability, and I doubt any ranking could get it right.

Even if we agreed on having no ranking (open question) there's no getting around the fact that the author list is an ordered list and that readers have preconceived notions about the meaning of that order. So we have to deal with it. Having the main writer listed as first author is, in my opinion, not always justified or necessary. It is perhaps even less so in this case, where the paper itself is a minor thing. Looking through a few JOSS papers I see that many use a symbol to indicate corresponding author (rather than placing them first) and a star to identify equally contributing authors.

The idea that is forming is to place Cliff first (perhaps as a substitute for the PVSC award) and the rest alphabetically, with Kevin identified as corresponding author.

If the present ranking (system) is maintained by majority vote, then I would ask that you add stars to the names with the explanation: "authors are sorted by number of commits" below the authors' names.

@kandersolar

Copy link
Copy Markdown
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants