Allow `delta_t` to be array in SPA functions

kandersolar commented 1 month ago

[x] Closes #2189
[x] I am familiar with the contributing guidelines
[x] Tests added
[ ] Updates entries in docs/sphinx/source/reference for API changes.
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for 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`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

See #2189. Additional tests are needed before this is ready for detailed review.

AdamRJensen commented 5 days ago

A better test instead of [65, 65] would be to have an array input with different values, e.g., [65, 66].

AdamRJensen commented 4 days ago

@kandersolar Thanks for the test update. I'll merge this tomorrow before the next release unless I hear any objections.

pvlib / pvlib-python