Skip to content

Replace all instances of OrderedDict with dict - #2855

Open
kandersolar wants to merge 2 commits into
pvlib:mainfrom
kandersolar:ordereddict
Open

Replace all instances of OrderedDict with dict#2855
kandersolar wants to merge 2 commits into
pvlib:mainfrom
kandersolar:ordereddict

Conversation

@kandersolar

Copy link
Copy Markdown
Member

Replacing all remaining instances of OrderedDict with regular dict. See #1684.

* :py:func:`~pvlib.irradiance.campbell_norman`
* :py:func:`~pvlib.irradiance.louche`
* :py:func:`~pvlib.pvsystem.sapm`
* :py:func:`~pvlib.pvsystem.max_power_point`

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the list of functions touched in this PR. I will combine it with similar whatsnew entries in the eventual release finalization PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

is there still a need for irradiance functions to return OrderDict since Python-3.7 is minimum version?

1 participant