Skip to content

Change OrderedDict to dict in perez and perez_driesse#2789

Open
cbcrespo wants to merge 1 commit into
pvlib:mainfrom
cbcrespo:perez-perez-driesse-components-dict
Open

Change OrderedDict to dict in perez and perez_driesse#2789
cbcrespo wants to merge 1 commit into
pvlib:mainfrom
cbcrespo:perez-perez-driesse-components-dict

Conversation

@cbcrespo

Copy link
Copy Markdown
Contributor
  • Closes #xxxx
  • I am familiar with the contributing guidelines
  • I attest that all AI-generated material has been vetted for accuracy and is in compliance with the pvlib license
  • Tests added
  • Updates entries in docs/sphinx/source/reference for API changes.
  • 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`).
  • New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
  • Pull request is nearly complete and ready for detailed review.
  • Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

This PR updates the output format of irradiance.perez and irradiance.perez_driesse when return_components=True, changing it from OrderedDict to a standard dict. The transition from OrderedDict to dict has been previously discussed in #1684.

As part of my GSoC 2026 project, I am modifying the diffuse transposition functions to ensure consistent support for the return_components parameter. During this work, I am also standardizing outputs by replacing remaining OrderedDict use with dict. perez and perez_driesse specifically do not require further changes beyond this update.

@kandersolar kandersolar added this to the v0.16.0 milestone Jun 18, 2026
@kandersolar

Copy link
Copy Markdown
Member

I think changing return type, even in such a minor way as this, is technically a breaking change, so I've added this to the v0.16.0 milestone.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants