172 update to latest version of pvlib#283
Merged
Merged
Conversation
Newer geokit narrows the accepted `points` types for gk.raster.interpolateValues to tuple/ogr.Geometry/Location/list/ LocationSet (numpy arrays and pandas Series are rejected), forwards **kwargs to extractValues in "near" mode, and returns a scalar for a single location. Adapt the reskit call sites accordingly: - topography: wrap the ndarray points in gk.LocationSet and drop the dead `_interpolation` parameter (was a no-op in old geokit, and its forwarded `interpolate` kwarg is now rejected by extractValues) - egs: wrap placement lon/lat ndarray in gk.LocationSet - csp dataset_handler: pass placements.geom as a list (.tolist()) instead of a pandas Series - logarithmic_profile: np.atleast_1d on the result so single-location scalar returns don't break item assignment - workflow_manager: np.atleast_1d on get_scalar_values_from_raster results so single-location placement groups (e.g. CSP per-dataset splits) return a 1-D array
…latest-version-of-pvlib
…in 0.10.0: Prevent small negative values of v_oc in pvlib.singlediode._lambertw() which result from accumulated roundoff error. ([GH1780](pvlib/pvlib-python#1780), [GH1673](pvlib/pvlib-python#1673), [GH1782](pvlib/pvlib-python#1782))
Collaborator
Author
chrisjwin
approved these changes
Jul 9, 2026
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
2 participants
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.
Currently pvlib is pinned to 0.9.0
In pvlib 0.10.0 values for module_dc_voltage_at_mpp change. This is due to a fix by pvlib in 0.10.0:
Prevent small negative values of v_oc in pvlib.singlediode._lambertw() which result from accumulated roundoff error. (pvlib/pvlib-python#1780, pvlib/pvlib-python#1673, pvlib/pvlib-python#1782)
That fix changes the single-diode MPP search slightly and shifts v_mp enough to move the interpolated module_dc_voltage_at_mpp regression value.
Other test values in our workflows are not affected.
As a consequence, we will need to adapt the slightly changed test values for module_dc_voltage_at_mpp: