Skip to content

Commit f20d78e

Browse files
committed
more max_rows behavior matching
1 parent 78da47f commit f20d78e

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

pvlib/bifacial/utils.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -243,8 +243,9 @@ def vf_ground_sky_2d_integ(surface_tilt, gcr, height, pitch, max_rows=10,
243243
collector_width = pitch * gcr
244244
surface_tilt = np.atleast_2d(np.abs(surface_tilt))
245245

246-
# TODO figure out if this range is correct, or if the original code has a bug
247-
k = np.arange(-max_rows+1, max_rows+1)[:, np.newaxis]
246+
# TODO seems like this should be np.arange(-max_rows, max_rows+1)?
247+
# see GH #1867
248+
k = np.arange(-max_rows, max_rows)[:, np.newaxis]
248249

249250
# primary crossed string points:
250251
# a, b: boundaries of ground segment

0 commit comments

Comments
 (0)