Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

ENH: model pvsyst thin-film recombination for CdTe and a:Si in pvsystem.singlediode() #517

Copy link
Copy link
Open
@mikofski

Description

@mikofski
Issue body actions

Problem
This request was initially made in #163 and is also related to #470. As of #504 bishop88 can accept the thin-film recombination parameters: d2mutau and vbi to output IV curve and its gradients, but there is no propagation of this ability to the proxies that use it such as pvsystem.singlediode()

Solution
Here is a proposed list of methods that should now be able to use these new terms:

  • singlediode_methods.bishop88_i_from_v()
  • singlediode_methods.bishop88_v_from_i()
  • singlediode_methods.bishop88_mpp()
  • singlediode_methods._lambertw_v_from_i()
  • singlediode_methods._lambertw_i_from_v()
  • singlediode_methods._lambertw()
  • pvsystem.PVSystem.singlediode()
  • pvsystem.PVSystem.i_from_v()
  • pvsystem.singlediode()
  • pvsystem.max_power_point()
  • pvsystem.v_from_i()
  • pvsystem.i_from_v()

Alternatives
Please suggest alternative solutions. There has been discussion of refactoring calcparams_pvsyst or even creating model specific singlediode_pvsyst methods. Is that desired?

Additional context
this is a follow on to #504

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Morty Proxy This is a proxified and sanitized view of the page, visit original site.