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

Fix: DPL limit is changing with every loop (solar-powered inverters) #1529

Conversation

AndreasBoehm
Copy link
Member

@AndreasBoehm AndreasBoehm commented Jan 13, 2025

Reverted as it leads to frequent, high changes of the limit on every other DPL loop.
Example: #1524

Original info that is true but can't be fixed with the rule that has been applied.

Low limits
Inverters with low limit (below 15%) are usually not able to provide the expected power and we would rate mppts as shaded.
This will result in us thinking that we can't get more power from that inverter and the limit is 'stuck'.

Also reverts #1507

Fixes #1530

@AndreasBoehm AndreasBoehm marked this pull request as draft January 14, 2025 09:08
@AndreasBoehm AndreasBoehm changed the title revert "Fix: Don't check shading for max increase when inverter limit is below 15% Fix: DPL limit is changing with every loop (solar-powered inverters) Jan 14, 2025
@AndreasBoehm AndreasBoehm force-pushed the andreasboehm/revert/solar-inverter-15-precent-maxincrease branch 2 times, most recently from 44d8f3c to db747a5 Compare January 15, 2025 14:48
… is below 15%"

Reverted as it leads to frequent high changes of the limit.
@AndreasBoehm AndreasBoehm force-pushed the andreasboehm/revert/solar-inverter-15-precent-maxincrease branch from db747a5 to 46fa993 Compare January 15, 2025 19:05
@AndreasBoehm AndreasBoehm marked this pull request as ready for review January 15, 2025 19:05
@schlimmchen schlimmchen merged commit 0a9a856 into development Jan 16, 2025
16 checks passed
@schlimmchen schlimmchen deleted the andreasboehm/revert/solar-inverter-15-precent-maxincrease branch January 16, 2025 20:01
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.

[DPL] with only the base load configured the limit is constantly changing
2 participants