Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Shaded Fraction on Sloped Terrains - PR1725 partial continuation (#1962)
* 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 * Apply suggestions from code review (Adam) Co-authored-by: Adam R. Jensen <[email protected]> * Update docs/examples/shading/plot_shaded_fraction1d_ns_hsat_example.py * Update docs/examples/shading/plot_shaded_fraction1d_ns_hsat_example.py --------- 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: Kevin Anderson <[email protected]>
- Loading branch information