Repository navigation
DOC: add reference callouts and enable footnote warnings - #2927
WhatIsAPM1989 wants to merge 3 commits into
Conversation
Connect existing references to the descriptions of the corresponding models, correct spectral model citations, and remove the global suppression of unreferenced-footnote warnings. No calculation code changes. Assisted-by: Codex (GPT-6)
Move callouts into extended descriptions so autosummary tables retain citation-free summaries and do not produce unknown-target errors. Assisted-by: Codex (GPT-6)
Assisted-by: Codex (GPT-6)
Hey @WhatIsAPM1989! 🎉Thanks for opening your first pull request! We appreciate your If AI is used for any portion of this PR, you must vet the content Finally, be sure the PR description includes the PR |
|
@WhatIsAPM1989 your AI PR stated that the draft version was reviewed by the agent and was awaiting your review. You marked the draft PR as ready for review now, but didn't remove that statement. You (or the AI?) also incorrectly ticked the maintainer item off the checklist when no labels or milestones have been added. Do not be offended if I am mistaken but this seems (at least to me) that no human oversight is present, and the AI attestation from the human contributor is intentionally false, so I am closing this PR. This PR could potentially be reopened if you can you confirm you reviewed this yourself, explain what that entailed, and fix the aforementioned issues. @pvlib/pvlib-maintainer please provide generalized guidance whether I should close PRs of this nature or offer guiding comments only. I did not review the diff given the contradictions in the PR; I am not convinced the guidelines are or will be followed. Feel free to re-open. |
|
My intention was to use my otherwise unused AI tokens to make a useful open-source contribution. I found pvlib through TOA (Tokens of Appreciation), which suggested this project and issue. The changes were prepared and reviewed by AI agents, but I did not personally review the diff or validate the technical content. I marked the PR ready without completing that human review, so the AI attestation and maintainer checkbox were incorrect. I apologize for the misleading checklist and description, and for taking up your time. |
|
Interesting. pvlib is # 4 on the “Climate” list here: https://donatetokens.org/#projects |
I'm in favor of coding agents, I use them myself. I support others using them too of course. However, I wonder how many people see the irony in wasting energy and water resources on AI slop framed as "donating" to a climate-related software tool 🫠 |
Do you call that AI slop because this code is really bad, or you call that any AI generated PR? |
|
Thanks for engaging. Your questions are fair.
Neither. It is because the guidelines weren't followed and no human oversight was provided at the initial opening, which is one of those guidelines. Lack of compliance in this basic way makes it difficult for reviewers to be convinced that the contributor will engage with the review process if they put the time into reviewing.
If you are willing to engage with the process, we absolutely welcome your contribution. If this is the case, let us know, fix the checklist, and I (and I'm sure others) will happily review your contribution:) |
|
Thank you for clarifying, and for being willing to review the contribution. I respect your guidelines. I don’t currently understand the technical material well enough to independently review and validate the changes myself, so I can’t provide the level of human oversight you require. Please leave the PR closed. I appreciate your time and the constructive explanation. |
Connect the existing bibliography entries to the corresponding model descriptions and remove the global suppression of unreferenced-footnote warnings. This also fixes the spectral-model citation mapping and adds the missing callouts in four gallery examples.
Validation: the HTML documentation build with gallery execution disabled (
-D plot_gallery=0) reduces unreferenced-footnote warnings from 99 to 0. The existing SPA autosummary andsingleaxismarkup diagnostics also reproduce on cleanmain, alongside offline inventory/example warnings. The CI-style Flake8 diff check passes, an AST comparison confirms that executable Python is unchanged, and all 702 rendered footnote links resolve to local targets.The earlier claim on #837 dates from January 2020; no current claim or competing PR was found.
docs/sphinx/source/whatsnew.AI disclosure: prepared autonomously with Codex (GPT-6) via TOA on behalf of @WhatIsAPM1989, self-reviewed by the agent. This draft awaits the contributor's review and AI-material attestation.