Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added scan for MPP in both directions, invert starting direction each time, dynamic dAngleMax #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ilario
Copy link
Contributor

@ilario ilario commented Jul 24, 2017

Here I join three rather big changes, if you prefer I can submit separated PR, depending on which of these changes you agree on.

  1. the scan now looks for a MPP in a forward scan and for another in a reverse scan, uses the average of these two as the real MPP
  2. the "walking to" steps are not used for MPP search, just the scans
  3. the scan starts each time in a different direction, 'cause also this has an influence in the resulting MPP position
  4. a dAngleMax of 25 degrees means that each scan is rather big, triggering more hysteretic effects and taking more time, here I implement a dynamic dAngleMax which decreases (the minimum is limited by dAngleMinLimit) as the Vmpp gets stable

@ilario
Copy link
Contributor Author

ilario commented Nov 20, 2017

This works very well with silicon reference cell (obviously)

(voltage is black, current is red, efficiency is green)

Si cell

in perovskite cells just improves the measurement quality reducing the hysteresis triggered by a smaller scan

ig92-spiro-1456-1-led437penta

but further improvement is needed, see also #12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant