Finalize 0.10.5 #2035
Merged
Finalize 0.10.5 #2035
Conversation
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
IoannisSifnaios
approved these changes
May 6, 2024
|
For the record, I believe the above review was performed by @AdamRJensen, just using @IoannisSifnaios's laptop :P I will merge this and proceed to make the release. Many thanks to all contributors! |
echedey-ls
pushed a commit
to echedey-ls/pvlib-python
that referenced
this pull request
May 22, 2024
* whatsnew cleanup * add v0.11.0 file * forgot to include myself in the contributors * update release date * Comments from review
echedey-ls
pushed a commit
to echedey-ls/pvlib-python
that referenced
this pull request
May 23, 2024
* whatsnew cleanup * add v0.11.0 file * forgot to include myself in the contributors * update release date * Comments from review
kandersolar
added a commit
that referenced
this pull request
Jun 21, 2024
* Update shading.py * Minimal test * Implementation From NREL paper * Fix, fix, fix, fix & format * Format issues * Extend tests (compare with singleaxis) & format with ruff * Format fixes * Upgrade tests * Array -> Axis * type * Whatsnew * xd * bruh * Minor Python optimization a la tracking.singleaxis * Comment and minor optimizations * Surface -> Axis Co-Authored-By: Kevin Anderson <[email protected]> * Elevation -> Zenith Co-Authored-By: Kevin Anderson <[email protected]> * Elev -> Zenith Co-Authored-By: Kevin Anderson <[email protected]> * Update shading.py * Update docstring Co-Authored-By: Anton Driesse <[email protected]> * Add comments from `tracking.singleaxis` Co-Authored-By: Will Holmgren <[email protected]> Co-Authored-By: Mark Mikofski <[email protected]> * Singleaxis implementation port & test addition, based on old pvlib.tracking.singleaxis * Update v0.10.4.rst * Linter * Code review Co-Authored-By: Cliff Hansen <[email protected]> * Add Fig 5 [1] (still gotta check the built output) * Add caption, change size and describe in alternate text * rST fixes ? * Figures have captions, images do not https://pandemic-overview.readthedocs.io/en/latest/myGuides/reStructuredText-Images-and-Figures-Examples.html#id18 * Flip arguments order * I forgot 💀 * Linter are you happy now? * Remove port test and add edge cases test Co-Authored-By: Kevin Anderson <[email protected]> * Update test_shading.py Co-Authored-By: Kevin Anderson <[email protected]> * Indentation xd * Update test_shading.py * I forgot how to code * Align data * Docstring suggestion from Kevin Co-Authored-By: Kevin Anderson <[email protected]> * Update link to example? * add linear shade loss for thin films * add tests, update docs, what's new * fix what's new gh issue and pr links * fix trailing whitespace * responding to comments - move linear shade loss to shading module - don't use ternary, doesn't work on vectors, instead use np.where() - set cross axis default to zero - test vectors - update docs * update docstring for linear shade loss - applicable to other monolithic thin film like CIGS, not just CdTe - only when shade is perpendicular to scribe lines * update example in linear_shade_loss * add figure and formulas to shaded fraction * shaded fraction consistently * Add alternative text to image * Update implementation based on PSZ PR. See description. Commit highlights ✨ : * I think I made all the code a bit more legible; sorry for the big changes @mikofski * Tests a bit more complete (not much, still consider the same test data) * Rename shaded fraction acronym from `fs` to `sf` * Asserts changed to `assert_allclose` for a more legible output in case of failure Co-Authored-By: Mark Mikofski <[email protected]> * Whatsnew entries Co-Authored-By: Mark Mikofski <[email protected]> * Linter * Clear things, convert Mark's reference to a reference * Linter * Update according to changes at PSZA PR * Another commit, another try * Ahhh, I rebased too fast * whatsnews * Update v0.10.4.rst * Update v0.10.3.rst * Rename to `shaded_fraction1d`, change params to `surface_*` instead of `tracker_*` * Left this tracker refs behind * Change rename in rst entries * Add another testcase * Improve docs references, clarify nomenclature Co-Authored-By: Kevin Anderson <[email protected]> * Update test_shading.py * Remove linear_shade_loss * First implementation of the new shaded fraction model (missing figure) * Create Anderson_Jensen_2024_Fig3.png * Update shading.py * Update shading.py * Update shading.py * lintaaargggg * Fill reference * Next release 0.10.5? * Fix tests * Update test_shading.py * Little improvement to table definitions * Change `l` to `\ell` Co-Authored-By: Kevin Anderson <[email protected]> * `pvlib.tracking.projected_solar_zenith_angle` to `pvlib.shading.projected_solar_zenith_angle` Co-Authored-By: Kevin Anderson <[email protected]> * pitch references to `pitch` Co-Authored-By: Kevin Anderson <[email protected]> * `trackers_axis_azimuth` to `axis_azimuth` Co-Authored-By: Kevin Anderson <[email protected]> * whatsnews Co-Authored-By: Kevin Anderson <[email protected]> * Update v0.10.5.rst Co-Authored-By: Kevin Anderson <[email protected]> * Change `tilt`s to `rotation`s and add `axis_tilt` Co-Authored-By: Kevin Anderson <[email protected]> * Forgot to update tests 💀 Co-Authored-By: Kevin Anderson <[email protected]> * Add examples section * roles assumption messin w/ me docs 😲 * roles assumption messin w/ me docs 😲 Co-Authored-By: Kevin Anderson <[email protected]> * Update shading.py * Update shading.py * Add gallery example * This was fixed in recent sphinx-gallery releases IIRC * Extra empty line or admonition type unsupported * Fix example link (hopefully 🙏 ) * Update shading.py * Fix subsubsections? * Nah, bulleted list didn't work * tilted -> tracker, only affects text * Typos and unreasonable physical values Co-Authored-By: Kevin Anderson <[email protected]> * See the Examples section below, not the unlinkable link Co-Authored-By: Kevin Anderson <[email protected]> * tracker -> row, param names, code and docs Co-Authored-By: Kevin Anderson <[email protected]> * Fix broken example 🔧 Co-Authored-By: Kevin Anderson <[email protected]> * Apply suggestions from code review Co-authored-by: Adam R. Jensen <[email protected]> * "the row axis/axes" instead of ``axis_azimuth`` * Unnecessary math mode Co-Authored-By: Adam R. Jensen <[email protected]> * Example suggestions and text trimming Co-Authored-By: Adam R. Jensen <[email protected]> * whatsmes * Add test to fix coverage issue * Initial work * Can't test with the paper data, wtf * Little things * Update plot_martinez_shade_loss.py * More improvements * Be4 rebase * Fix fixture * Initial work * Fix tests * docstring * whatsnew 📝 * Minor thingies * Add figure, improve docs 💯 * Fix silently ignoring file path in `pvsystem.retrieve_sam` when `name` is provided (#2020) * My approach to the issue * Deprecate previous parameters * No reason to over-engineer, right? * Update v0.10.5.rst * Update pvsystem.py * Improve error handling * Add ppl involved * kevin's suggestions * Fix Ixx equation in pvsystem.sapm (#2019) * fix Ixx equation, use Aimp * Update docs/sphinx/source/whatsnew/v0.10.5.rst * lint * lint * get the spacing right * more spacing --------- Co-authored-by: Kevin Anderson <[email protected]> * Increase python requirement to >= 3.8 (#2029) * require python >=3.8 * update CI configurations * whatsnew * remove pre-3.8 cruft * fix h5py/numpy versioning issue for py3.8-min * update gallery examples for newer pandas * update asv config for python 3.8 * fix asv config syntax error * one more asv config edit * Finalize 0.10.5 (#2035) * whatsnew cleanup * add v0.11.0 file * forgot to include myself in the contributors * update release date * Comments from review * update whatsnews * Update test_shading.py * Example * Fix whatsmes * Update v0.11.0.rst * other img * lintarrrrrr 😭 * docs * docs refurbishment * Update plot_martinez_shade_loss.py * Fix parameter names * More explicit 2D shaded fraction Co-Authored-By: Kevin Anderson <[email protected]> * Explicitier explicit Co-Authored-By: Kevin Anderson <[email protected]> * Update shading.py Co-Authored-By: Kevin Anderson <[email protected]> * Only if life was VCS to revert my errors so easily Co-Authored-By: Kevin Anderson <[email protected]> * Nix image in docstring Co-Authored-By: Kevin Anderson <[email protected]> * Clear-up irrdiance loss, not pwr loss - yet again Co-Authored-By: Kevin Anderson <[email protected]> * rewording Co-Authored-By: Kevin Anderson <[email protected]> * Remove figure of junction boxes in example Co-Authored-By: Kevin Anderson <[email protected]> * Duplicated singleaxis call Co-Authored-By: Kevin Anderson <[email protected]> * When did I change this? Co-Authored-By: Kevin Anderson <[email protected]> * Rename function Co-Authored-By: Kevin Anderson <[email protected]> * Make times just one statement Co-Authored-By: Kevin Anderson <[email protected]> * Fix uncomplete example in docstring Co-Authored-By: Kevin Anderson <[email protected]> * flake is a snowflake ❄️ Co-Authored-By: Kevin Anderson <[email protected]> * Delete Centralized_and_split_PV_junction_boxes_cesardd.jpg Co-Authored-By: Kevin Anderson <[email protected]> * Remove example description Co-Authored-By: Cliff Hansen <[email protected]> * Change example title Co-Authored-By: Cliff Hansen <[email protected]> * Ints for number of blocks Co-Authored-By: Cliff Hansen <[email protected]> * Rephrasing of shaded fraction 2D Co-Authored-By: Cliff Hansen <[email protected]> * typo * Code review from Ioannis Co-Authored-By: Ioannis Sifnaios <[email protected]> * no more noqa e501 * Will this work to hide code? * Update plot_martinez_shade_loss.py Co-Authored-By: Ioannis Sifnaios <[email protected]> * play time with the matplotlib statements Co-Authored-By: Ioannis Sifnaios <[email protected]> * This fixes it (I believe 🙏 ) CC @IoannisSifnaios Co-Authored-By: Ioannis Sifnaios <[email protected]> * Update plot_martinez_shade_loss.py Co-Authored-By: Ioannis Sifnaios <[email protected]> * Nah, let's see how this does Co-Authored-By: Ioannis Sifnaios <[email protected]> * Revert attempt to colapse code It doesn't work because sphinx-gallery closes the figure at the end of the cell sphinx-gallery/sphinx-gallery#240 CC @IoannisSifnaios Co-Authored-By: Ioannis Sifnaios <[email protected]> * Power losses model Co-Authored-By: Kevin Anderson <[email protected]> Co-Authored-By: Cliff Hansen <[email protected]> Co-Authored-By: Ioannis Sifnaios <[email protected]> * equations rendering * Change `poa_direct_and_circumsolar` to `poa_direct` Co-Authored-By: Kevin Anderson <[email protected]> * Example link Co-Authored-By: Kevin Anderson <[email protected]> * `power_loss` -> `loss_fraction` Co-Authored-By: Kevin Anderson <[email protected]> * Typo when applying Ioannis suggestion Co-Authored-By: Kevin Anderson <[email protected]> Co-Authored-By: Ioannis Sifnaios <[email protected]> * Re-arrange POA to its own section in example Co-Authored-By: Kevin Anderson <[email protected]> * Mis-redaction in normal/half-cut modules comparison Co-Authored-By: Kevin Anderson <[email protected]> * Add edge cases Co-Authored-By: Kevin Anderson <[email protected]> * Link yet again Co-Authored-By: Kevin Anderson <[email protected]> * Fix link for sure Co-Authored-By: Kevin Anderson <[email protected]> * Remove critical error from docs build * Apply suggestions from code review (Cliff) Co-authored-by: Cliff Hansen <[email protected]> * Linter * Apply suggestions from Adam Co-authored-by: Adam R. Jensen <[email protected]> * Rewording from code review Co-Authored-By: Adam R. Jensen <[email protected]> * More unitless Co-Authored-By: Adam R. Jensen <[email protected]> * Apply suggestions from Cliff Co-authored-by: Cliff Hansen <[email protected]> --------- Co-authored-by: Kevin Anderson <[email protected]> Co-authored-by: Anton Driesse <[email protected]> Co-authored-by: Will Holmgren <[email protected]> Co-authored-by: Mark Mikofski <[email protected]> Co-authored-by: Cliff Hansen <[email protected]> Co-authored-by: Adam R. Jensen <[email protected]> Co-authored-by: Cliff Hansen <[email protected]> Co-authored-by: Kevin Anderson <[email protected]> Co-authored-by: Ioannis Sifnaios <[email protected]>
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.
[ ] Tests added[ ] Updates entries indocs/sphinx/source/referencefor API changes.docs/sphinx/source/whatsnewfor all changes. Includes link to the GitHub Issue with:issue:`num`or this Pull Request with:pull:`num`. Includes contributor name and/or GitHub username (link with:ghuser:`user`).[ ] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.remote-data) and Milestone are assigned to the Pull Request and linked Issue.