Update test suite for pvlib v0.9.0 #115
Merged
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.
Description
This PR updates the test suite to not use the
orientation_strategy
ModelChain parameter (removed in v0.9.0). Here is an example failed CI run from #114: https://github.com/pvlib/pvanalytics/runs/4398633530?check_suite_focus=trueAt some point we will need to stop using
SingleAxisTracker
as well, but it still works for now.Checklist
The following items must be addressed before the code can be merged.
Please don't hesitate to ask for help if you are unsure of how to accomplish any of the items.
You are free to remove any checklist items that do not apply or add additional items that are
not on this list
Closes #xxxAdded new API functions todocs/api.rst
Clearly documented all new API functions with PEP257 and numpydoc compliant docstringsin
docs/whatsnew
for all changes. Includes link to the GitHub Issue with
:issue:`num`
or this Pull Request with
:pull:`num`
. Includes contributor nameand/or GitHub username (link with
:ghuser:`user`
).Non-API functions clearly documented with docstrings or comments as necessaryAdded tests to cover all new or modified code